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(Form): fix Field onChange reset value #2197
base: main
Are you sure you want to change the base?
Conversation
@MarvelSQ is attempting to deploy a commit to the rsuite Team on Vercel. A member of the Team first needs to authorize it. |
This pull request is being automatically deployed with Vercel (learn more). rsuite-nextjs – ./docs🔍 Inspect: https://vercel.com/rsuite/rsuite-nextjs/44k5PrTWXG8ZPA6Drt89QsddVAsY |
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 a2d2bd6:
|
Codecov Report
@@ Coverage Diff @@
## main #2197 +/- ##
==========================================
+ Coverage 85.21% 85.24% +0.03%
==========================================
Files 269 269
Lines 8493 8497 +4
Branches 2356 2356
==========================================
+ Hits 7237 7243 +6
Misses 685 685
+ Partials 571 569 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
|
||
const _formValueRef = useRef(_formValue); | ||
|
||
_formValueRef.current = _formValue; |
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.
这解决方案,很奇怪,并且可读性不太好,我更推荐使用 useReducer
fix #2192