Skip to content
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: avoid flashing when ou levels are used (DHIS2-17136) #3032

Merged
merged 2 commits into from
May 8, 2024

Conversation

edoardo
Copy link
Member

@edoardo edoardo commented Apr 24, 2024

Part of the fixes for DHIS2-17136


Key features

  1. fix flashing of visualization

Description

This only happened when ou levels were selected for the ou dimension.
The fix avoids that the visualization object passed to the ChartPlugin is a different one every time.

@edoardo edoardo requested a review from jenniferarnesen April 24, 2024 13:47
@dhis2-bot
Copy link
Contributor

dhis2-bot commented Apr 24, 2024

@dhis2-bot dhis2-bot temporarily deployed to netlify April 24, 2024 13:50 Inactive
Copy link

cypress bot commented Apr 24, 2024

Passing run #2812 ↗︎

0 769 2 0 Flakiness 0

Details:

Merge 02b87bf into 827ef37...
Project: Data Visualizer App Commit: 2d323ac32f ℹ️
Status: Passed Duration: 06:14 💡
Started: Apr 26, 2024 12:22 PM Ended: Apr 26, 2024 12:28 PM

Review all test suite changes for PR #3032 ↗︎

@dhis2-bot dhis2-bot temporarily deployed to netlify April 25, 2024 10:55 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify April 26, 2024 06:50 Inactive
@edoardo edoardo force-pushed the fix/plugin-flashing-resize branch from 8b058da to 27916b2 Compare April 26, 2024 10:05
@dhis2-bot dhis2-bot temporarily deployed to netlify April 26, 2024 10:08 Inactive
@edoardo edoardo force-pushed the fix/plugin-flashing-resize branch from 27916b2 to 02b87bf Compare April 26, 2024 10:17
@dhis2-bot dhis2-bot temporarily deployed to netlify April 26, 2024 10:20 Inactive
@edoardo edoardo changed the title fix: avoid flashing when ou levels are used fix: avoid flashing when ou levels are used (DHIS2-17136) Apr 26, 2024
@dhis2-bot dhis2-bot temporarily deployed to netlify May 8, 2024 12:05 Inactive
@edoardo edoardo merged commit b1b6148 into dev May 8, 2024
20 checks passed
@edoardo edoardo deleted the fix/plugin-flashing-resize branch May 8, 2024 13:23
@cypress cypress bot mentioned this pull request May 13, 2024
dhis2-bot added a commit that referenced this pull request May 13, 2024
## [100.5.4](v100.5.3...v100.5.4) (2024-05-13)

### Bug Fixes

* avoid flashing when ou levels are used (DHIS2-17136) ([#3032](#3032)) ([b1b6148](b1b6148))
* fix a crash when row totals is enabled (DHIS2-17297) ([#3042](#3042)) ([4004455](4004455))
* **translations:** sync translations from transifex (dev) ([f1a14b8](f1a14b8))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 100.5.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

kabaros pushed a commit that referenced this pull request May 30, 2024
## [100.5.4](v100.5.3...v100.5.4) (2024-05-13)

### Bug Fixes

* avoid flashing when ou levels are used (DHIS2-17136) ([#3032](#3032)) ([b1b6148](b1b6148))
* fix a crash when row totals is enabled (DHIS2-17297) ([#3042](#3042)) ([4004455](4004455))
* **translations:** sync translations from transifex (dev) ([f1a14b8](f1a14b8))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants