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: export function for preparing payload for copy (DHIS2-15722) #1573

Merged
merged 2 commits into from
Sep 25, 2023

Conversation

edoardo
Copy link
Member

@edoardo edoardo commented Sep 8, 2023

Part of the fix for DHIS2-15722

Relates to dhis2/line-listing-app#433


Key features

  1. export function for preparing the payload for visualization copy (FileMenu -> Save As)

Description

The bug described in the ticket applies to all analytics apps, all need to use the same fix.
Instead of duplicating the code in all apps, export a utility function from analytics that can be used by all apps.

Better than relying on apps not fetching the user information when
loading a visualization.
@edoardo edoardo force-pushed the fix/function-for-preparing-ao-for-copy-DHIS2-15722 branch from f62754b to 327ce6b Compare September 15, 2023 13:21
@edoardo edoardo merged commit a1f8d69 into master Sep 25, 2023
2 checks passed
@edoardo edoardo deleted the fix/function-for-preparing-ao-for-copy-DHIS2-15722 branch September 25, 2023 13:28
dhis2-bot added a commit that referenced this pull request Sep 25, 2023
## [26.0.20](v26.0.19...v26.0.20) (2023-09-25)

### Bug Fixes

* export function for preparing payload for copy DHIS2-15722 ([#1573](#1573)) ([a1f8d69](a1f8d69))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 26.0.20 🎉

The release is available on:

Your semantic-release bot 📦🚀

@janhenrikoverland janhenrikoverland changed the title fix: export function for preparing payload for copy DHIS2-15722 fix: export function for preparing payload for copy (DHIS2-15722) Oct 20, 2023
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