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: use translated dimension labels in PT (DHIS2-15750) #1572

Merged
merged 1 commit into from
Sep 15, 2023

Conversation

edoardo
Copy link
Member

@edoardo edoardo commented Sep 6, 2023

Fixes DHIS2-15750

Relates to https://github.com/dhis2/data-visualizer-app/pull/XXX


Key features

  1. use translated strings for dimension labels in PT

Description

In Pivot Table visualizations it's possible to enable dimension labels via options.
These were not translated.
The PR fixes the issue by marking those strings for translation.


Screenshots

Before:
Screenshot 2023-09-06 at 12 00 10

Screenshot 2023-09-06 at 12 12 52 Screenshot 2023-09-06 at 12 12 33

After:
Screenshot 2023-09-06 at 12 00 29

Screenshot 2023-09-06 at 12 00 47 Screenshot 2023-09-06 at 12 11 46

@edoardo edoardo force-pushed the fix/translate-dimension-labels-pt-DHIS2-15750 branch from 3b70090 to a341e57 Compare September 14, 2023 14:15
@edoardo edoardo merged commit 492975e into master Sep 15, 2023
2 checks passed
@edoardo edoardo deleted the fix/translate-dimension-labels-pt-DHIS2-15750 branch September 15, 2023 07:24
dhis2-bot added a commit that referenced this pull request Sep 15, 2023
## [26.0.19](v26.0.18...v26.0.19) (2023-09-15)

### Bug Fixes

* use translated dimension labels in PT DHIS2-15750 ([#1572](#1572)) ([492975e](492975e))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 26.0.19 🎉

The release is available on:

Your semantic-release bot 📦🚀

@janhenrikoverland janhenrikoverland changed the title fix: use translated dimension labels in PT DHIS2-15750 fix: use translated dimension labels in PT (DHIS2-15750) 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