Skip to content

Resolve custom data elements on Funder#1457

Merged
martha merged 1 commit into
release-212from
mke/9165-daily-rate
May 4, 2026
Merged

Resolve custom data elements on Funder#1457
martha merged 1 commit into
release-212from
mke/9165-daily-rate

Conversation

@martha

@martha martha commented May 1, 2026

Copy link
Copy Markdown
Contributor

Description

Summary of changes: The Funder schema object already has custom data elements in the API, but we weren't previously resolving them, so a saved CDE wouldn't appear correctly in the form.

Relates to hmis-warehouse PR: greenriver/hmis-warehouse#6432 (no dependency, can be merged independently)

How to test: Save a CDE on a Funder record, then reopen the form. The CDE should be populated in the form.

Type of change

Fix to a non-presenting bug (no existing Funders collect CDEs)

Checklist before requesting review

  • I have performed a self-review of my code
  • I have run the code that is being changed under ideal conditions, and it doesn't fail
  • I have used Axe DevTools to scan for accessibility issues (or not applicable)
  • My code includes comments and/or descriptive variable names to help other engineers understand the intent (or not applicable)
  • My code follows the style guidelines of this project (eslint)
  • I have updated the documentation (or not applicable)
  • If it's not obvious how to test this change, I have provided testing instructions in this PR or the related issue

@martha martha changed the title resolve custom data elements on funder Resolve custom data elements on Funder May 1, 2026
@martha martha marked this pull request as ready for review May 1, 2026 22:02
@martha martha requested a review from gigxz May 1, 2026 22:02
@martha martha merged commit b4e1c1c into release-212 May 4, 2026
4 checks passed
@martha martha deleted the mke/9165-daily-rate branch May 4, 2026 11:53
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