-
Notifications
You must be signed in to change notification settings - Fork 15
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: infinite spinner when opening the interpretations modal while viewing a visualization #3283
Conversation
🚀 Deployed on https://pr-3283.data-visualizer.netlify.dhis2.org |
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.
Please change how the default value for onResponsesReceived
is set. Apart from that everything looks good, so I will approve already,
src/components/VisualizationPlugin/VisualizationPluginWrapper.js
Outdated
Show resolved
Hide resolved
So essentially this PR fixes a bug in which I introduced when working on DHIS2-16793, and @jenniferarnesen asked me to double check that the current PR did not cause any problems with the original bugfix. I can hereby confirm it doesn't, see below: |
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.
We've discussed the implementation in detail and I verified that the original bug is still solved. APPROVED.
## [100.8.4](v100.8.3...v100.8.4) (2024-11-07) ### Bug Fixes * infinite spinner when opening the interpretations modal while viewing a visualization ([#3283](#3283)) ([403adbc](403adbc))
🎉 This PR is included in version 100.8.4 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Fixes DHIS2-18369
The InterpretationsModal callback onResponsesReceived was not being called after a recent fix introduced its own onResponsesReceived callback
TODO