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

docs: use named component export and improve qiankun config #10848

Merged
merged 2 commits into from
Mar 24, 2023

Conversation

fz6m
Copy link
Contributor

@fz6m fz6m commented Mar 24, 2023

  1. 主应用访问微应用的时候,微应用服务掉线,主应用会报错,请问我该怎么捕获这个错误?并处理他 #10824 反馈发现,现在无法在配置文件里使用 tsx ,序列化写文件的过程中函数都会被清除掉,故删除 qiankun 在配置文件中使用 errorBoundary 的部分。

  2. 所有匿名函数改为具名函数,防止误导导致 react refresh 不生效。

@vercel
Copy link

vercel bot commented Mar 24, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
umi ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 24, 2023 at 0:55AM (UTC)

@codecov
Copy link

codecov bot commented Mar 24, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (8c671d2) 29.25% compared to head (d251ce8) 29.25%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #10848   +/-   ##
=======================================
  Coverage   29.25%   29.25%           
=======================================
  Files         469      469           
  Lines       14196    14196           
  Branches     3340     3340           
=======================================
  Hits         4153     4153           
  Misses       9334     9334           
  Partials      709      709           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@sorrycc sorrycc merged commit 7109cef into umijs:master Mar 24, 2023
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.

2 participants