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: close TranslationDialog after saving a translation #2245

Merged
merged 1 commit into from
Mar 15, 2023

Conversation

edoardo
Copy link
Member

@edoardo edoardo commented Mar 15, 2023

This is the same behaviour as in the apps.
The dialog closing has been recently changed so it needs to be triggered in the app.

This is the same behaviour as in the apps.
@dhis2-bot
Copy link
Contributor

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

@dhis2-bot dhis2-bot temporarily deployed to netlify March 15, 2023 11:00 Inactive
@edoardo edoardo merged commit 295acb3 into master Mar 15, 2023
@edoardo edoardo deleted the fix/close-translation-dialog-on-save branch March 15, 2023 15:59
dhis2-bot added a commit that referenced this pull request Mar 15, 2023
## [99.10.17](v99.10.16...v99.10.17) (2023-03-15)

### Bug Fixes

* close TranslationDialog after saving a translation ([#2245](#2245)) ([295acb3](295acb3))
* unwanted re-rendering of dashboard items ([#2247](#2247)) ([b02714a](b02714a))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 99.10.17 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

janhenrikoverland pushed a commit that referenced this pull request Mar 17, 2023
This is the same behaviour as in the apps.
jenniferarnesen pushed a commit that referenced this pull request Mar 22, 2023
This is the same behaviour as in the apps.
jenniferarnesen added a commit that referenced this pull request Mar 23, 2023
This is the same behaviour as in the apps.

Co-authored-by: Edoardo Sabadelli <[email protected]>
Co-authored-by: Jan Henrik Øverland <[email protected]>
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