-
-
Notifications
You must be signed in to change notification settings - Fork 49.8k
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: Update rc-motion
deps & fix ssr
#26542
Conversation
@@ -129,7 +129,7 @@ | |||
"rc-input-number": "~6.0.0", | |||
"rc-mentions": "~1.4.0", | |||
"rc-menu": "~8.5.2", | |||
"rc-motion": "^1.0.0", | |||
"rc-motion": "^1.1.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里用的是 ^,可能已经影响了之前的 antd 版本。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rc-motion
出来的时间不长,这次重构主要是改成 FC 解决 SSR 的问题。对过去影响不会很大。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PS: 发 minor 也是因为重构,没有添加新特性。
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 860e607:
|
Codecov Report
@@ Coverage Diff @@
## master #26542 +/- ##
=======================================
Coverage 99.75% 99.75%
=======================================
Files 383 383
Lines 7343 7343
Branches 2003 2051 +48
=======================================
Hits 7325 7325
Misses 18 18
Continue to review full report at Codecov.
|
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
fix #26519
💡 Background and solution
📝 Changelog
help
make ssr un-sync issue.help
时出现的同构问题。☑️ Self Check before Merge