Skip to content

Conversation

natalialuzuriaga
Copy link
Contributor

@natalialuzuriaga natalialuzuriaga commented Sep 10, 2025

Problem

Currently, the web form does not use the latest v2 Gov and CMS Schema. We also would like to improve the instructions and guidance on required vs optional fields

Solution

  • Update Gov & CMS schema to the latest version: v2.0.0
  • Update form instructions with link to SHARE IT Act Landing Page
  • Add optional text to optional fields as recommended here: https://designsystem.digital.gov/components/form/
  • Add helper text explaining required fields as recommended here: https://designsystem.digital.gov/components/form/
  • Update meta tag to include description content
  • Added a check for empty objects to identify whether an optional field is blank (removes from code.json if so)

Test Plan

Works locally python3 -m http.server 8000 , successfully generates CMS and Gov code.json

Signed-off-by: Natalia Luzuriaga <[email protected]>
Signed-off-by: Natalia Luzuriaga <[email protected]>
Signed-off-by: Natalia Luzuriaga <[email protected]>
Signed-off-by: Natalia Luzuriaga <[email protected]>
Signed-off-by: Natalia Luzuriaga <[email protected]>
Signed-off-by: Natalia Luzuriaga <[email protected]>
Signed-off-by: Natalia Luzuriaga <[email protected]>
Copy link
Contributor

@DinneK DinneK left a comment

Choose a reason for hiding this comment

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

Pulled locally, LGTM! Small note for the future, not a blocker: the button class="usa-button usa-button--secondary formio-button-remove-row shows up as a red box with no indication for what it's for. It looks like this is a upstream Form.io issue, but should be put on the list as a future fix.

@natalialuzuriaga
Copy link
Contributor Author

Good catch, documented upstream here! formio/uswds#268

@natalialuzuriaga natalialuzuriaga merged commit 89f9520 into main Sep 11, 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