Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

VATEAM-90732: Add OMB info fields to Digital Form #19078

Merged
merged 1 commit into from
Sep 5, 2024

Conversation

derekhouck
Copy link
Contributor

@derekhouck derekhouck commented Aug 28, 2024

Description

Adds all fields necessary to render an accurate OMB info component on each form. Specifically, adds the following two fields:

  • Expiration date (string, will be converted to expected pattern in content-build)
  • Respondent Burden (integer, in minutes)

These two fields are grouped together with OMB Number to form an "OMB Info" field group.

Testing done

Updated the Digital Form bundle class test with the new fields. Tested the form and view displays locally.

Screenshots

The "OMB info" field group in the form display:
Screenshot 2024-08-28 at 2 27 51 PM

The "OMB info" field group in the node display:
Screenshot 2024-08-28 at 2 29 27 PM

QA steps

As an administrator

  1. Go to Manage > Content > Add content > Digital Form
    • Verify the "OMB info" field group is visible and the following fields are contained within it:
      • "Respondent burden"
      • "OMB Number"
      • "Expiration date"
  2. Verify all three of the above fields are required.
  3. Enter data in the "Respondent burden" field
    • Verify you can only enter numbers.
    • Verify you can only enter numbers greater than or equal to 0.
  4. Select a date in the "Expiration date" field.
    • Verify it displays the date in MM/DD/YYYY format.
  5. Fill out the other required fields on the Digital Form and click "Save" at the bottom of the page.
    • Verify the Digital Form saves successfully and the view page looks similar to the above screenshot.

Definition of Done

  • Documentation has been updated, if applicable.
  • Tests have been added if necessary.
  • Automated tests have passed.
  • Code Quality Tests have passed.
  • Acceptance Criteria in related issue are met.
  • Manual Code Review Approved.
  • If there are field changes, front end output has been thoroughly checked.

Select Team for PR review

  • CMS Team
  • Public websites
  • Facilities
  • User support
  • Accelerated Publishing

Is this PR blocked by another PR?

  • DO NOT MERGE

Does this PR need review from a Product Owner

  • Needs PO review

CMS user-facing announcement

Is an announcement needed to let editors know of this change?

  • Yes, and it's written in issue ____ and queued for publication.
    • Merge and ping the UX writer so they are ready to publish after deployment
  • Yes, but it hasn't yet been written
    • Don't merge yet -- ping the UX writer to write and queue content
  • No announcement is needed for this code change.
    • Merge & carry on unburdened by announcements

@derekhouck derekhouck requested a review from ryguyk August 28, 2024 19:52
@derekhouck derekhouck marked this pull request as ready for review August 28, 2024 19:52
@github-actions github-actions bot added the CMS Team CMS Product team that manages both editor exp and devops label Aug 28, 2024
Copy link
Contributor

@ryguyk ryguyk left a comment

Choose a reason for hiding this comment

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

Looks great!

I do have some lingering questions about how configuration like this will be presented/honored/etc. in our custom form experience, but I do think this work will be useful regardless.

@derekhouck
Copy link
Contributor Author

I do have some lingering questions about how configuration like this will be presented/honored/etc. in our custom form experience, but I do think this work will be useful regardless.

If we can tie our custom display into Drupal's display configuration system, that would be ideal, but I don't know off-hand how much work that would be at this time.

@ryguyk
Copy link
Contributor

ryguyk commented Sep 3, 2024

If we can tie our custom display into Drupal's display configuration system, that would be ideal, but I don't know off-hand how much work that would be at this time.

We can, at least to some degree. But there's still the question of how things map from the default display to our custom display. For example, does a fieldset become its own page? Or, perhaps to ask it better, since we will have designs for each page of the wizard-like experience, do we define fieldsets accordingly?

In other words, we should be able to access the display configuration, but there are some questions in my head about what we should do with that information or how what that means for how we configure it in the first place.

@ryguyk ryguyk force-pushed the VAGOV-TEAM-87958-integration-form-engine-poc branch from 5c55049 to af73e95 Compare September 3, 2024 18:10
@ryguyk ryguyk requested review from a team as code owners September 3, 2024 18:10
@ryguyk ryguyk force-pushed the VAGOV-TEAM-87958-integration-form-engine-poc branch from af73e95 to 2c479a4 Compare September 4, 2024 13:53
@ryguyk ryguyk force-pushed the 90732-create-omb-info-section branch from 11a16b1 to 259fc9d Compare September 4, 2024 14:59
@va-cms-bot va-cms-bot temporarily deployed to Tugboat September 4, 2024 14:59 Destroyed
@ryguyk ryguyk changed the base branch from VAGOV-TEAM-87958-integration-form-engine-poc to VAGOV-TEAM-87958-integration-form-engine-2 September 4, 2024 14:59
@ryguyk ryguyk changed the base branch from VAGOV-TEAM-87958-integration-form-engine-2 to integration-form-engine-form-21-4140 September 4, 2024 16:56
@ryguyk ryguyk deleted the branch integration-form-engine-2 September 4, 2024 17:00
@ryguyk ryguyk closed this Sep 4, 2024
@ryguyk ryguyk reopened this Sep 4, 2024
@va-cms-bot va-cms-bot temporarily deployed to Tugboat September 4, 2024 17:02 Destroyed
@ryguyk ryguyk changed the base branch from integration-form-engine-form-21-4140 to integration-form-engine-2 September 4, 2024 17:02
@ryguyk ryguyk force-pushed the 90732-create-omb-info-section branch from 259fc9d to a8feb41 Compare September 5, 2024 18:49
@va-cms-bot va-cms-bot temporarily deployed to Tugboat September 5, 2024 18:50 Destroyed
@ryguyk ryguyk merged commit e4cd6f2 into integration-form-engine-2 Sep 5, 2024
12 of 16 checks passed
@ryguyk ryguyk deleted the 90732-create-omb-info-section branch September 5, 2024 18:56
@ryguyk ryguyk mentioned this pull request Sep 13, 2024
21 tasks
ryguyk pushed a commit that referenced this pull request Sep 16, 2024
ryguyk pushed a commit that referenced this pull request Sep 17, 2024
* Add OMB info fields to Digital Form (#19078)

* Add Identification Information pattern

* Add OMB Info to digital-forms script

* Add Identification Information to digital-forms script
@derekhouck derekhouck restored the 90732-create-omb-info-section branch October 2, 2024 16:37
@va-cms-bot va-cms-bot temporarily deployed to Tugboat October 2, 2024 16:37 Destroyed
derekhouck added a commit that referenced this pull request Oct 2, 2024
* Add OMB info fields to Digital Form (#19078)

* Add Identification Information pattern

* Add OMB Info to digital-forms script

* Add Identification Information to digital-forms script
ryguyk pushed a commit that referenced this pull request Dec 12, 2024
* Add OMB info fields to Digital Form (#19078)

* Add Identification Information pattern

* Add OMB Info to digital-forms script

* Add Identification Information to digital-forms script
ryguyk added a commit that referenced this pull request Dec 12, 2024
* VATEAM-90582: Add 21-4140 Digital Form to seed script (#19047)

* Add create_digital_forms

* Add 21-4140 to digital forms script

* Switch includeDob to snake case

* VATEAM-90628: Add Identification Information Paragraph type (#19190)

* Add OMB info fields to Digital Form (#19078)

* Add Identification Information pattern

* Add OMB Info to digital-forms script

* Add Identification Information to digital-forms script

* VATEAM-90650: Create Address Digital Form Pattern (#19331)

Creates a Paragraph type in Drupal for the "Address" pattern. The Paragraph type includes an "Include military address checkbox?" boolean field. Adds the new Paragraph type to the list of available Paragraph types in the "Steps" field of the Digital Form content type.

* VATEAM-90651: Create Phone and Email Digital Form pattern (#19464)

Creates a Paragraph type in Drupal for the "Phone and Email Address" pattern. The Paragraph type includes an "Include email address?" boolean field. Adds the new Paragraph type to the list of available Paragraph types in the "Steps" field of the Digital Form content type.

* VATEAM-94366: Create "Your personal information" Paragraph type (#19534)

Create a new Paragraph type called "Your personal information" that contains the "Name and Date of Birth" and "Identification Information" Paragraph types within it. Remove "Name and Date of Birth" and "Identification Information" as selectable Steps. Update the Digital Forms script to use the new Paragraph type.

* VATEAM-90652: Create List & Loop pattern (#19816)

Creates a Paragraph type in Drupal for the "List & Loop" pattern. The Paragraph type includes an "Optional?" boolean field. Adds the new Paragraph type to the list of available Paragraph types in the "Steps" field of the Digital Form content type.

---------

Co-authored-by: Derek Houck <[email protected]>
Co-authored-by: Derek Houck <[email protected]>
Co-authored-by: Derek Houck <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CMS Team CMS Product team that manages both editor exp and devops
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants