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.

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

Fix: dashboard error #911

Merged
merged 1 commit into from
Jul 21, 2021
Merged

Fix: dashboard error #911

merged 1 commit into from
Jul 21, 2021

Conversation

luhc228
Copy link
Contributor

@luhc228 luhc228 commented Jul 19, 2021

Closed: #857

问题:在外网环境下,获取项目信息(基本信息和依赖信息)失败

  • 获取依赖信息失败原因是使用 latest-version 获取依赖最新版本太慢了
  • 外网环境下获取 def 信息时有请求内网地址,但没有设置超时时间

解决:

  • 现在改用 package-json (支持传入 registry 地址)获取依赖最新版本
  • 判断是否在外网环境下,如果是,则不去获取 def 信息

@luhc228 luhc228 requested a review from fyangstudio July 19, 2021 09:36
@fyangstudio fyangstudio merged commit c0e7bc3 into release/1.3.0 Jul 21, 2021
@fyangstudio fyangstudio deleted the fix/dashboard-error branch July 21, 2021 02:29
This was referenced Jul 21, 2021
fyangstudio added a commit that referenced this pull request Jul 22, 2021
* fix: override.files neo work

* feat: update

* Feat/prop types auto complete (#902)

* feat: complete proptypes

* feat: code snippets (#903)

* feat: auto fill content in store's files

* docs: update changelog (#909)

* refactor: variable rename

* fix: dashboard error in external (#911)

* feat: import auto complete (#905)

* feat: import-auto-complete

* feat: export types (#904)

* feat: import auto completion  (#914)

* feat: import auto complete

* feat: update release (#913)

* fix: code snippets (#915)

* fix: code snippets

* feat: update registry

Co-authored-by: GiveMe-A-Name <58852732+GiveMe-A-Name@users.noreply.github.com>
Co-authored-by: Hengchang Lu <44047106+luhc228@users.noreply.github.com>
Co-authored-by: GiveMe-A-Name <a455300764b@hotmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants