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: apply legend to all numeric and boolean types (DHIS2-17611) #1683

Merged
merged 7 commits into from
Jun 26, 2024

Conversation

janhenrikoverland
Copy link
Collaborator

@janhenrikoverland janhenrikoverland commented Jun 18, 2024

Implements DHIS2-17611


Description

This PR does two things:

  • Applies legends to all numeric and boolean values, not just value type NUMBER, to support event data items
  • Renders booleans as numeric values (integers), ie .0 is stripped so that the number of "Yes" is presented as 3 instead of 3.0

TODO

  • Manual testing
  • Figure out the removed i18n

Screenshots

Before
Screenshot from 2024-06-18 15-43-06

After
Screenshot from 2024-06-18 15-43-10

(The negative integer value is just out of bounds of the legend set)

@janhenrikoverland janhenrikoverland changed the title fix: apply legend to all numeric and boolean types fix: apply legend to all numeric and boolean types (DHIS2-17611) Jun 19, 2024
Copy link
Member

@edoardo edoardo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is an extra output.tgz file which shouldn't end up in master.

src/components/PivotTable/PivotTableValueCell.js Outdated Show resolved Hide resolved
i18n/en.pot Outdated Show resolved Hide resolved
@janhenrikoverland janhenrikoverland merged commit e93b106 into master Jun 26, 2024
5 checks passed
@janhenrikoverland janhenrikoverland deleted the fix/pivot-tables-legend-value-type branch June 26, 2024 14:26
dhis2-bot added a commit that referenced this pull request Jun 26, 2024
## [26.7.6](v26.7.5...v26.7.6) (2024-06-26)

### Bug Fixes

* apply legend to all numeric and boolean types (DHIS2-17611) ([#1683](#1683)) ([e93b106](e93b106))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 26.7.6 🎉

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
Development

Successfully merging this pull request may close these issues.

None yet

3 participants