Skip to content

feat: [DHIS2-12881] Support additional value types for filters in working lists#4461

Open
henrikmv wants to merge 89 commits intomasterfrom
hv/feat/DHIS2-12881_AddFilterForAllTypes
Open

feat: [DHIS2-12881] Support additional value types for filters in working lists#4461
henrikmv wants to merge 89 commits intomasterfrom
hv/feat/DHIS2-12881_AddFilterForAllTypes

Conversation

@henrikmv
Copy link
Contributor

@henrikmv henrikmv commented Feb 15, 2026

DHIS2-12881

This pull request introduces filtering support for value types:

  • AGE
  • COORDINATE
  • DATETIME
  • EMAIL
  • FILE_RESOURCE
  • IMAGE
  • LONG_TEXT
  • ORGANISATION_UNIT
  • PERCENTAGE
  • PHONE_NUMBER
  • TIME
  • URL
  • USERNAME

devin-ai-integration bot and others added 7 commits February 10, 2026 10:43
…st filter

Co-Authored-By: henrik.vadet@dhis2.org <henrik@devOtta.no>
Call getSearchOperator() in _setBaseProperties() so that program
attributes used by working lists carry the correct searchOperator.
This fixes both issues:
- preferredSearchOperator was always undefined in working lists
- unique TEAs were not using EQ operator (getSearchOperator handles this)

Co-Authored-By: henrik.vadet@dhis2.org <henrik@devOtta.no>
Co-Authored-By: henrik.vadet@dhis2.org <henrik@devOtta.no>
@henrikmv henrikmv requested a review from a team as a code owner February 15, 2026 20:04
Copy link
Contributor

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

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 5 additional findings.

Open in Devin Review

devin-ai-integration[bot]

This comment was marked as resolved.

@henrikmv henrikmv marked this pull request as draft February 16, 2026 15:53
@henrikmv henrikmv added e2e-tests e2e record Apply this label to a pull request to trigger recording of E2E tests on Cypress Cloud and removed testing e2e-tests e2e record Apply this label to a pull request to trigger recording of E2E tests on Cypress Cloud labels Mar 5, 2026
@henrikmv henrikmv added e2e-tests e2e record Apply this label to a pull request to trigger recording of E2E tests on Cypress Cloud and removed e2e-tests e2e record Apply this label to a pull request to trigger recording of E2E tests on Cypress Cloud labels Mar 5, 2026
@henrikmv henrikmv added testing and removed e2e-tests e2e record Apply this label to a pull request to trigger recording of E2E tests on Cypress Cloud labels Mar 5, 2026
@henrikmv henrikmv requested a review from simonadomnisoru March 5, 2026 14:23
@henrikmv henrikmv marked this pull request as ready for review March 5, 2026 14:24
devin-ai-integration[bot]

This comment was marked as resolved.

henrikmv and others added 2 commits March 5, 2026 15:52
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
devin-ai-integration[bot]

This comment was marked as resolved.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 5, 2026

@henrikmv henrikmv added testing and removed testing labels Mar 6, 2026
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