-
Notifications
You must be signed in to change notification settings - Fork 14k
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: Reset sorting bar issue in Barchart #19371
fix: Reset sorting bar issue in Barchart #19371
Conversation
Codecov Report
@@ Coverage Diff @@
## master #19371 +/- ##
==========================================
- Coverage 66.59% 66.53% -0.06%
==========================================
Files 1670 1672 +2
Lines 63888 63954 +66
Branches 6510 6519 +9
==========================================
+ Hits 42544 42552 +8
- Misses 19655 19711 +56
- Partials 1689 1691 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
LGTM!
* fix: Reset sorting bar issue in Barchart * improving performance * fix lint error
SUMMARY
When enabling "sort bars" and disabling again, the order isn't reset to the original order.
It was because when we sort data from props, it made updates in redux store as well.
Solution: deep copy
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
BEFORE:
[DEV] Genders by State - 25 March 2022 - Watch Video
AFTER:
[DEV] Genders by State - 25 March 2022 - Watch Video
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION