zthxxx / react-dev-inspector

jump to local IDE code directly from browser React component by just a simple click
https://react-dev-inspector.zthxxx.me
MIT License
1.13k stars 67 forks source link

功能需求: umi4 支持 [v1.8.6 已支持] #132

Closed xiefengnian closed 1 year ago

xiefengnian commented 1 year ago

react-dev-inspector/plugins/umi/react-inspector 需要支持 umi4

zthxxx commented 1 year ago

@xiefengnian 没用 umi4,有空看看; 或者你来提个 PR 不?🌝

xiefengnian commented 1 year ago

@xiefengnian 没用 umi4,有空看看; 或者你来提个 PR 不?🌝

搞一个 plugins/umi4 ?

JamieMing commented 1 year ago

看了代码,github上的源码应该是支持umi4的,但是npm上的产物代码并不一致,是直接调用了modifyBabelOpts,但umi4取消了这个方法,而作者github代码上是已经做了判断处理的。希望作者可以更新一下npm上的代码~

JamieMing commented 1 year ago

我现在是直接将react-inspector的代码拉了一份出来,在plugins配置中引入,并去掉参数设置(因为会报错),即可

zthxxx commented 1 year ago

@xiefengnian @JamieMing 有 PR 合并了兼容 umi4 的代码,之前发版失败了,现在已经重新发到 v1.8.2 的版本上了

Yyunfan commented 1 year ago

@zthxxx 哈喽,plugins\umi\react-inspector.js#L50, umi的api拼错了,应该是addBeforeMiddlewares

zthxxx commented 1 year ago

@Yyunfan 这个 typo 是旧版 umi 自己提供 api 就写错了的、umi 自己提供向下兼容的;

现在会导致你使用有报错吗?什么版本,多提供点信息

Yyunfan commented 1 year ago

@zthxxx 用的 @umijs/max 的4.0.22版本。react-dev-inspector 是最新1.8.2。umi那边会报 TypeError: api.addBeforeMiddewares is not a function

zthxxx commented 1 year ago

@Yyunfan 这个 api 加了兼容处理了,发到 1.8.3 版本了

Yyunfan commented 1 year ago

@zthxxx 修复了typo错误后,仍然会报错,TypeError: Cannot read property 'register' of undefined,貌似这次更新还影响到隔壁ant-design-pro库的使用,也会报这个错,大概的错误堆栈是这个 at service.pluginMethods.<computed>.fn (\node_modules\@umijs\core\dist\service\pluginAPI.js:115:14)

ado5 commented 1 year ago

期待更新

ado5 commented 1 year ago

vite4 什么时候能也支持一下?

baikaifa commented 1 year ago

这个解决完了吗?我尝试了还是不行,文档上要不补充下umi4怎么用。

zthxxx commented 1 year ago

发版了,试试 v1.8.6

尝试支持了 umi4 的 addBeforeBabelPlugins api

Yyunfan commented 1 year ago

ok,umi4确实可以使用了 👍