Skip to content

fix: text payload for validationRules expression mutation#1420

Merged
KaiVandivier merged 1 commit intomasterfrom
LIBS-815/validationRule-expression-payload
Dec 17, 2025
Merged

fix: text payload for validationRules expression mutation#1420
KaiVandivier merged 1 commit intomasterfrom
LIBS-815/validationRule-expression-payload

Conversation

@tomzemp
Copy link
Copy Markdown
Member

@tomzemp tomzemp commented Dec 17, 2025

Implements LIBS-815


Key features

Makes mutations to api/validationRules/expression/description send text/plain payload


Description

Logic in app-runtime determines the appropriate payload type, content headers for mutations. We have, from before, logic to use plain text payload with api/indicators/expression/description and api/programIndicators/description, but we missed api/validationRules/expression/description


Checklist

  • Have written Documentation
    • We do not document this logic in our descriptions of mutations.
  • [ X ] Has tests coverage
    • unit tests are updated for this endpoint

@tomzemp tomzemp requested a review from a team December 17, 2025 09:59
@sonarqubecloud
Copy link
Copy Markdown

@KaiVandivier KaiVandivier merged commit da955b3 into master Dec 17, 2025
18 checks passed
@KaiVandivier KaiVandivier deleted the LIBS-815/validationRule-expression-payload branch December 17, 2025 14:46
dhis2-bot added a commit that referenced this pull request Dec 17, 2025
## [3.14.7](v3.14.6...v3.14.7) (2025-12-17)

### Bug Fixes

* use content type text/plain for mutations to validationRules/expression/description ([#1420](#1420)) ([da955b3](da955b3))
@dhis2-bot
Copy link
Copy Markdown
Contributor

kabaros pushed a commit to tibendadavis/app-runtime that referenced this pull request Dec 30, 2025
kabaros pushed a commit to tibendadavis/app-runtime that referenced this pull request Dec 30, 2025
* use content type text/plain for mutations to validationRules/expression/description ([dhis2#1420](dhis2#1420)) ([da955b3](dhis2@da955b3))
kabaros pushed a commit to tibendadavis/app-runtime that referenced this pull request Dec 30, 2025
* use content type text/plain for mutations to validationRules/expression/description ([dhis2#1420](dhis2#1420)) ([da955b3](dhis2@da955b3))
kabaros pushed a commit to tibendadavis/app-runtime that referenced this pull request Dec 30, 2025
* use content type text/plain for mutations to validationRules/expression/description ([dhis2#1420](dhis2#1420)) ([da955b3](dhis2@da955b3))
kabaros added a commit that referenced this pull request Dec 30, 2025
* feat: added a useCurrentUserInfo hook to encapsulate getting user info

* feat: add useCurrentUserInfo hook tests and related documentation

* refactor: remove UserContext and related hook, streamline UserProvider

* refactor: remove unused useMemo import from UserProvider

* feat: update useCurrentUserInfo documentation and type definitions

and replace CurrentUserState with CurrentUser type

* fix: ensure UserContext defaults to defaultUser when userInfo is undefined

* refactor: remove DHIS2_PROXY_URL from start script and clean up dependencies

* refactor: clean up code formatting and improve readability in various files

* refactor: clean up

* docs: update dhis2.nu links

* docs: styles

* fix: use content type text/plain for mutations to validationRules/expression/description (#1420)

* chore(release): cut 3.14.7 [skip release]

* refactor: update README to reflect correct service name

* fix: make userInfo prop optional in ProviderInput type

* fix: improved userInfo to be of type currentUser or undefined in UserProvider component

* test: remove unused defaultUser object and update expectation for undefined userInfo

* fix: update userInfo prop to require CurrentUser or undefined in ProviderInput type

* fix: format code for consistency in UserProvider component

* refactor: update the yarn.lock

* chore: revert adding user-service to runtime until it is published

* chore: fix lint errors

* chore(release): cut 3.14.7 [skip release]

* use content type text/plain for mutations to validationRules/expression/description ([#1420](#1420)) ([da955b3](da955b3))

---------

Co-authored-by: Kai Vandivier <kpvandivier@gmail.com>
Co-authored-by: Thomas Zemp <tom@dhis2.org>
Co-authored-by: @dhis2-bot <apps@dhis2.org>
Co-authored-by: Mozafar Haider <mozafar@dhis2.org>
dhis2-bot added a commit that referenced this pull request Jan 5, 2026
## [3.14.8](v3.14.7...v3.14.8) (2026-01-05)

### Bug Fixes

* rerun pipeline ([0593525](0593525))
* revert yarn.lock ([#1426](#1426)) ([5d4ebde](5d4ebde))

### Features

* add useCurrentUserInfo hook ([#1417](#1417)) ([e5cbc63](e5cbc63)), closes [#1420](#1420)
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.

3 participants