Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

开源 @umijs/request-record 与开启机制优化相关问题 #11

Open
leshalv opened this issue Dec 17, 2022 · 8 comments
Open

开源 @umijs/request-record 与开启机制优化相关问题 #11

leshalv opened this issue Dec 17, 2022 · 8 comments
Labels
bug Something isn't working

Comments

@leshalv
Copy link

leshalv commented Dec 17, 2022

image
@leshalv
Copy link
Author

leshalv commented Dec 17, 2022

应该是 @umijs/request-record 依赖造成的

@chenshuai2144 chenshuai2144 added the bug Something isn't working label Dec 21, 2022
@xiefengnian
Copy link

应该是 @umijs/request-record 依赖造成的

需要提供一个复现方法

@mashirozx
Copy link

+1， 用的最新umi max

@leshalv leshalv mentioned this issue Jan 5, 2023
Closed
@leshalv
Copy link
Author

leshalv commented Jan 5, 2023

配置上 requestRecord 就好了，但是会生成相应文件，如果不配置这个功能，是要报错的。

@leshalv
Copy link
Author

leshalv commented Jan 5, 2023

umijs/umi#10053

@fz6m
Copy link
Member

fz6m commented Jan 5, 2023

经排查，@umijs/request-record 这个插件的开启机制是 proxy 的有无，只要配了 proxy 就默认开启，建议优化成 requestRecord: {} 这个选项是否开启。

另外把 @umijs/request-record 这个开源出来吧。

目前源码可以在 这里 看到。

@fz6m fz6m changed the title 2.0.0版本 fatal - TypeError: Cannot read properties of undefined (reading 'mock') 开源 @umijs/request-record 与开启机制优化相关问题 Jan 5, 2023
@huangjihua
Copy link

pro 新构建的项目，登录404，有人说注释 proxy里dev:{}, 也不管用

@huangjihua
Copy link

image
pnpm install ，并么有安装@@/requestRecordMock这个包，配置都是taobao 源，什么原因？

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@huangjihua @chenshuai2144 @mashirozx @xiefengnian @leshalv @fz6m