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: event/enrollment ui changes to prepare for TEI cross-program support (DHIS2-15454) #415

Merged
merged 60 commits into from
Oct 25, 2023

Conversation

martinkrulltott
Copy link
Contributor

@martinkrulltott martinkrulltott commented Jul 12, 2023

Implements DHIS2-15454 + DHIS2-15530

Figma

Spec


Key features

  1. Major ui changes to selection of program, stage and "main dimensions"
  2. Includes fix from fix: programs that the user doesn't have data access to are shown (DHIS2-15530) #409

Description

The ui gone through a redesign for program, stage and dimensions selection to support more input types in the future. All the details can be found in the figma sketch and design spec (links above), but this basically means that both program/stage selection and what used to be called "main dimensions" have moved to create a better user flow.


TODO

  • Cypress tests
  • Update docs
  • Design review with Joe
  • KFMT
  • fix broken unit tests
  • clear everything when switching input or program

Known issues

  • remove temporary filter once the backend works properly again (see TODO in YourDimensionsPanel.js)
  • fileMenu.cy.js tests fail because its relying on clearing stage and program (which isn't possible any more)

Screenshots

TBC, pending final design review

@dhis2-bot
Copy link
Contributor

dhis2-bot commented Jul 12, 2023

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

@dhis2-bot dhis2-bot temporarily deployed to netlify July 12, 2023 13:03 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify July 13, 2023 12:49 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify July 13, 2023 13:08 Inactive
@martinkrulltott martinkrulltott self-assigned this Jul 14, 2023
@dhis2-bot dhis2-bot temporarily deployed to netlify July 14, 2023 12:51 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify August 7, 2023 11:18 Inactive
@martinkrulltott martinkrulltott marked this pull request as ready for review August 9, 2023 08:12
@dhis2-bot dhis2-bot temporarily deployed to netlify August 9, 2023 08:20 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify August 9, 2023 08:42 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify August 10, 2023 07:59 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify October 6, 2023 09:15 Inactive
@janhenrikoverland janhenrikoverland changed the title feat: event/enrollment ui changes to prepare for TEI cross-program support feat: event/enrollment ui changes to prepare for TEI cross-program support (DHIS2-15454) Oct 19, 2023
@dhis2-bot dhis2-bot temporarily deployed to netlify October 19, 2023 08:48 Inactive
program is an object that changes at each render triggering the fetch
every time.
Instead, use a memoized program.id which is the only thing needed in the
fetch.
Memoize also setIsListEndVisible to avoid returning a difference
function from the hook.
@dhis2-bot dhis2-bot temporarily deployed to netlify October 20, 2023 08:44 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify October 23, 2023 10:02 Inactive
@martinkrulltott martinkrulltott requested review from edoardo and removed request for a team October 23, 2023 10:07
@dhis2-bot dhis2-bot temporarily deployed to netlify October 24, 2023 14:14 Inactive
Copy link
Collaborator

@janhenrikoverland janhenrikoverland left a comment

Choose a reason for hiding this comment

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

Approved when image10.png has been updated.

@dhis2-bot dhis2-bot temporarily deployed to netlify October 25, 2023 08:14 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify October 25, 2023 08:21 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify October 25, 2023 08:27 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify October 25, 2023 09:32 Inactive
@martinkrulltott martinkrulltott merged commit 53fa071 into master Oct 25, 2023
17 checks passed
@martinkrulltott martinkrulltott deleted the feat/DHIS2-15454-sidebar-ui-changes branch October 25, 2023 13:24
dhis2-bot added a commit that referenced this pull request Oct 26, 2023
# [100.10.0](v100.9.5...v100.10.0) (2023-10-26)

### Features

* event/enrollment ui changes to prepare for TEI cross-program support (DHIS2-15454) ([#415](#415)) ([53fa071](53fa071))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 100.10.0 🎉

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants