Skip to content

Form Bug Fixes#92

Merged
natalialuzuriaga merged 4 commits intomainfrom
DSACMS/nat/bug-fixes
Aug 20, 2025
Merged

Form Bug Fixes#92
natalialuzuriaga merged 4 commits intomainfrom
DSACMS/nat/bug-fixes

Conversation

@natalialuzuriaga
Copy link
Copy Markdown
Contributor

Problem

Addressed some bug fixes as well as schema updates

Solution:

  • Update gov schema and CMS schema to latest and aligned form components accordingly
  • Properties with an array value used to be assigned as a dictionary but now are correctly assigned as an array
  • When creating JSON object, skip / do not not assign optional properties with blank values

Test Plan

tested locally

Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
…ordingly

Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Copy link
Copy Markdown
Contributor

@decause-gov decause-gov left a comment

Choose a reason for hiding this comment

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

LGTM +1 🚢 🪨

Rock Solid Nat. We'll figure out some way to indicate "yes, I would like to be included in the AI Use Case Inventory" even if they are not there yet, but this works for now. Likely, using a tag like "AIUseCaseCandidate" or something like that.

@natalialuzuriaga natalialuzuriaga merged commit 781dfa6 into main Aug 20, 2025
4 of 5 checks passed
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