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: main dimensions not translated in line lists (DHIS2-15878) #502

Merged
merged 4 commits into from
Apr 24, 2024

Conversation

martinkrulltott
Copy link
Contributor

@martinkrulltott martinkrulltott commented Apr 16, 2024

Implements DHIS2-15878


Key features

  1. Allow all main dimensions to be translated, for all input types

Description

Some main dimensions were previously not translated properly, as they would fall back to the column field in the response header instead of using custom metadata. This caused the table header to display non-translated labels (even though the layout and dimension lists were translated properly).

(🟢 = previously broken, now fixed. 🔵 = was already working, 🟡 = separate ticket)

Input: Event

  • 🟢 Organisation unit
  • 🟢 Event status
  • 🟢 Program status
  • 🟢 Last updated on
  • 🟢 Created by
  • 🟢 Last updated by
  • 🟡 Default time dimensions
  • 🔵 Custom time dimensions (not translated)

Input: Enrollment

  • 🟢 Organisation unit
  • 🟢 Program status
  • 🟢 Last updated on
  • 🟢 Created by
  • 🟢 Last updated by
  • 🟡 Default time dimensions
  • 🔵 Custom time dimensions (not translated)

Input: Tracked Entity

  • 🔵 Organisation unit
  • 🔵 Program status
  • 🟢 Registration org. unit
  • 🟢 Registration date
  • 🟢 Last updated on
  • 🟢 Created by
  • 🟢 Last updated by
  • 🟡 Default time dimensions
  • 🔵 Custom time dimensions (not translated)

TODO

  • Cypress tests - will be added at a later stage
  • Update docs - not needed
  • Manual testing

Known issues

  • Table header translations aren't working on a dashboard
  • Table header translations aren't working for default time dimension labels - solved by separate ticket DHIS2-17265

Screenshots

event

image

enrollment

image

tracked entity

image

@martinkrulltott martinkrulltott requested a review from a team as a code owner April 16, 2024 13:59
@dhis2-bot
Copy link
Contributor

dhis2-bot commented Apr 16, 2024

🚀 Deployed on https://pr-502--dhis2-line-listing.netlify.app

@dhis2-bot dhis2-bot temporarily deployed to netlify April 16, 2024 14:01 Inactive
Copy link

cypress bot commented Apr 16, 2024

Passing run #2677 ↗︎

0 693 3 0 Flakiness 0

Details:

Merge faed264 into 8139e89...
Project: line-listing-app Commit: d724276ac5 ℹ️
Status: Passed Duration: 13:31 💡
Started: Apr 24, 2024 2:55 PM Ended: Apr 24, 2024 3:08 PM

Review all test suite changes for PR #502 ↗︎

@dhis2-bot dhis2-bot temporarily deployed to netlify April 22, 2024 10:28 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify April 22, 2024 11:39 Inactive
@janhenrikoverland
Copy link
Collaborator

@martinkrulltott creates a new ticket for time dimensions.

@dhis2-bot dhis2-bot temporarily deployed to netlify April 24, 2024 14:50 Inactive
@martinkrulltott martinkrulltott merged commit d3639b2 into master Apr 24, 2024
33 checks passed
@martinkrulltott martinkrulltott deleted the fix/DHIS2-15878-main-dimension-translations branch April 24, 2024 15:33
dhis2-bot added a commit that referenced this pull request Apr 24, 2024
## [100.16.2](v100.16.1...v100.16.2) (2024-04-24)

### Bug Fixes

* main dimensions not translated in line lists (DHIS2-15878) ([#502](#502)) ([d3639b2](d3639b2))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 100.16.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

4 participants