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): check dashboard id before calling redux methods #15921
Conversation
e7b0ea1
to
8b92971
Compare
Codecov Report
@@ Coverage Diff @@
## master #15921 +/- ##
==========================================
+ Coverage 76.91% 76.98% +0.07%
==========================================
Files 986 986
Lines 51993 51995 +2
Branches 7090 7091 +1
==========================================
+ Hits 39988 40030 +42
+ Misses 11779 11739 -40
Partials 226 226
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Thanks for the quick diagnostic and fix! |
…he#15921) * check dashboard id before calling redux methods * comment
…he#15921) * check dashboard id before calling redux methods * comment
…he#15921) * check dashboard id before calling redux methods * comment
…he#15921) * check dashboard id before calling redux methods * comment
SUMMARY
Fixes an issue caused by dashboards using
UNSAFE_componentWillReceiveProps
to make changes in redux. When switching dashboards via SPA, this check doesn't work. Need to check the dashboard id.BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TESTING INSTRUCTIONS
Go to the Birth Names dashboard, then navigate to the World Health dashboard. A warning toast should not pop up.
ADDITIONAL INFORMATION