Skip to content

fix: [DHIS2-18620][DHIS2-21937] Hidden TEA is displayed in the TEI profile - #4704

Open
henrikmv wants to merge 34 commits into
masterfrom
hv/fix/DHIS2-18620_hidden-TEA-displayed-in-TEI-profile
Open

fix: [DHIS2-18620][DHIS2-21937] Hidden TEA is displayed in the TEI profile#4704
henrikmv wants to merge 34 commits into
masterfrom
hv/fix/DHIS2-18620_hidden-TEA-displayed-in-TEI-profile

Conversation

@henrikmv

@henrikmv henrikmv commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

This PR fixes two bugs:

DHIS2-18620 — Run program rules at the enrollment scope (new useEnrollmentScopeRuleEffects) and have WidgetProfile subscribe to rulesEffectsHiddenFields to filter its display list.

DHIS2-21937 — Derive programOwnerId from the enrollment and resolve the org unit via useCoreOrgUnit(programOwnerId) instead of the URL, so the modal loads when no org unit is selected.


Note: DHIS2-18620 breaks WidgetProfile's self-contained pattern (now depends on rulesEffectsHiddenFields populated externally). Tracked for a proper revamp in DHIS2-21954.

@github-actions

Copy link
Copy Markdown

devin-ai-integration[bot]

This comment was marked as resolved.

@henrikmv
henrikmv marked this pull request as ready for review August 18, 2026 13:26
@henrikmv
henrikmv requested a review from a team as a code owner August 18, 2026 13:26
@henrikmv henrikmv added testing and removed testing labels Aug 18, 2026
@henrikmv henrikmv added testing and removed testing labels Aug 18, 2026
…t-infinite-loading-without-org-unit' into hv/fix/DHIS2-18620_hidden-TEA-displayed-in-TEI-profile
…le-widget-infinite-loading-without-org-unit' into hv/fix/DHIS2-18620_hidden-TEA-displayed-in-TEI-profile"

This reverts commit 2054d1b, reversing
changes made to 45c023f.
@henrikmv henrikmv added testing and removed testing labels Aug 18, 2026
@henrikmv
henrikmv marked this pull request as draft August 18, 2026 14:29
devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@henrikmv henrikmv added testing and removed testing labels Aug 19, 2026
@henrikmv henrikmv removed the testing label Aug 19, 2026

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Devin Review found 1 new potential issue.

Open in Devin Review

@henrikmv henrikmv added testing and removed testing labels Aug 26, 2026
@henrikmv
henrikmv marked this pull request as ready for review August 26, 2026 19:43
@henrikmv henrikmv added testing and removed testing labels Aug 26, 2026
@henrikmv henrikmv added testing and removed testing labels Aug 27, 2026

@simonadomnisoru simonadomnisoru left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice! 🥳

@henrikmv henrikmv added testing and removed testing labels Sep 3, 2026
@henrikmv henrikmv added testing and removed testing labels Sep 3, 2026
@henrikmv henrikmv added testing and removed testing labels Sep 3, 2026
@sonarqubecloud

sonarqubecloud Bot commented Sep 8, 2026

Copy link
Copy Markdown

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.

2 participants