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: fetch visualization always when caching (DHIS2-17509) v38 #2988

Merged
merged 1 commit into from
Jun 5, 2024

Conversation

edoardo
Copy link
Member

@edoardo edoardo commented Jun 4, 2024

Fixes DHIS2-17509


Key features

  1. backport of fix: fetch visualization always when caching (DHIS2-17509) #2986

Description

A previous fix for an item flashing issue caused the offline cache to lack the request for the visualizations.
The fix looks at the recording state and triggers a fetch that can be recorded.

(cherry picked from commit c1201fa)

A previous fix for an item flashing issue caused the offline cache to
lack the request for the visualizations.
The fix looks at the recording state and triggers a fetch that can be
recorded.

(cherry picked from commit c1201fa)
@edoardo edoardo requested a review from jenniferarnesen June 4, 2024 13:04
@dhis2-bot
Copy link
Contributor

🚀 Deployed on https://pr-2988--dhis2-dashboard.netlify.app

@dhis2-bot dhis2-bot temporarily deployed to netlify June 4, 2024 13:12 Inactive
Copy link

cypress bot commented Jun 4, 2024

Passing run #4867 ↗︎

0 59 0 0 Flakiness 0

Details:

fix: fetch visualization always when caching
Project: dashboards-app Commit: 7c4bb8a13b
Status: Passed Duration: 05:00 💡
Started: Jun 4, 2024 1:15 PM Ended: Jun 4, 2024 1:20 PM

Review all test suite changes for PR #2988 ↗︎

@edoardo edoardo changed the title fix: fetch visualization always when caching v38 (DHIS2-17509) fix: fetch visualization always when caching (DHIS2-17509) v38 Jun 4, 2024
@janhenrikoverland janhenrikoverland merged commit 0445704 into v38 Jun 5, 2024
11 checks passed
@janhenrikoverland janhenrikoverland deleted the backport/visualization-offline-cache-v38 branch June 5, 2024 11:44
janhenrikoverland pushed a commit that referenced this pull request Jun 5, 2024
A previous fix for an item flashing issue caused the offline cache to
lack the request for the visualizations.
The fix looks at the recording state and triggers a fetch that can be
recorded.

(cherry picked from commit c1201fa)
janhenrikoverland added a commit that referenced this pull request Jun 5, 2024
A previous fix for an item flashing issue caused the offline cache to
lack the request for the visualizations.
The fix looks at the recording state and triggers a fetch that can be
recorded.

(cherry picked from commit c1201fa)

Co-authored-by: Edoardo Sabadelli <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants