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

feat: implement option set selector in Data dimension modal (DHIS2-17872) #3344

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

edoardo
Copy link
Member

@edoardo edoardo commented Feb 17, 2025

Implements DHIS2-17872

Requires dhis2/analytics#1746


Key features

  1. implement "options view mode" for selecting options for data items with assigned option set

Description

See description in the analytics PR.

Can be tested against the https://dev.im.dhis2.org/pdeo-pao instance as it requires backend support which is not yet merged in master.
Run locally with yarn start --app --proxy=https://dev.im.dhis2.org/pdeo-pao and login using http://localhost:8080 as instance URL.


TODO

  • Cypress tests
  • Update docs
  • Manual testing
  • task

Known issues

  • missing optionSetId in the data returned by the visualization api (solved by extracting the optionSet id from dataDimensionItems)
  • the options list does not filter by data item due to missing backend support, adding that filter causes the request to fail

Screenshots

Screenshot 2025-02-17 at 16 03 52

@dhis2-bot
Copy link
Contributor

dhis2-bot commented Feb 17, 2025

🚀 Deployed on https://pr-3344.data-visualizer.netlify.dhis2.org

@dhis2-bot dhis2-bot temporarily deployed to netlify February 17, 2025 15:28 Inactive
This information is not returned as part of columns/rows/filters but it
can be extracted from dataDimensionItems.
The optionSetId is also added to the metadata when fetching the
dataItems in the Data modal.
The rest of the code relies on the presence of optionSetId in metadata
in order to enable the option view mode.
@dhis2-bot dhis2-bot temporarily deployed to netlify February 18, 2025 11:20 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify February 18, 2025 13:58 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify February 18, 2025 14:09 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify February 18, 2025 14:30 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify February 19, 2025 10:30 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify February 19, 2025 11:09 Inactive
@edoardo edoardo marked this pull request as ready for review February 19, 2025 14:40
@dhis2-bot dhis2-bot temporarily deployed to netlify February 20, 2025 13:25 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify February 21, 2025 12:38 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants