Skip to content
This repository is currently being migrated. It's locked while the migration is in progress.

v56.3.0#2057

Merged
powellkerry merged 2 commits intomainfrom
v56.3.0
Mar 30, 2026
Merged

v56.3.0#2057
powellkerry merged 2 commits intomainfrom
v56.3.0

Conversation

@powellkerry
Copy link
Copy Markdown
Contributor

@powellkerry powellkerry commented Mar 30, 2026

Change Log:

Component Summary API Change Rendered markup change
va-file-input, va-file-input-multiple Updates va-file-input and va-file-input-multiple to support the existing behavior for encrypted file passwords (event emission on change of nested va-text-input for password) and the new pattern that includes a submit button for the inputted password. (PR: #2027, Author: RyanMunsch) Yes Yes
Dependency update Bumps flatted from 3.3.3 to 3.4.1 in the npm_and_yarn group. (PR: #2033, Author: dependabot) No No
Component guidance tags Backfills guidanceHref on top-level web components and adds lint enforcement to prevent future omissions. (PR: #2036, Author: humancompanion-usds) No No
va-file-input-multiple Storybook Removes stringified passwordSubmissionSuccessList state value in the EncryptedTemplate from rendered output. I was using this value to debug template validation logic in #2027. (PR: #2042, Author: RyanMunsch) No Yes
va-telephone-input Updates the va-telephone-input component to be compatible with the form-system such that errors are properly set on the component. (PR: #2028, Author: it-harrison) No Yes
CSS-Library: mega-menu Changes mega-menu overflow from scroll to auto to remove unnecessary scrollbar behavior. (PR: #2048, Author: powellkerry) No Yes
va-date, va-file-input-multiple, va-file-input, va-memorable-date, va-telephone-input, va-text-input Makes changes to various interactive input components to normalize label bold styles based on the components' error states. Labels should be bold when there is an error present. (PR: #2041, Author: RyanMunsch) No Yes
Revert - Stencil/puppeteer upgrade Reverts stencil and puppeteer upgrade due to runtime/modal issues, with follow-up tracked separately. (PR: #2049, Author: powellkerry) No No
va-on-this-page Update va-on-this-page to add a prop to exclude h2s that match heading selectors. Update utils to add a function to normalize string arrays so that the react binding is not necessary to use the prop (PR: #2043, Author: powellkerry) Yes Yes

Testing and review

Provide any testing instructions or review steps as needed.

Approvals

See the QA Checklists section below for suggested approvals. Use your best judgment if additional reviews are needed. When in doubt, request a review.

Approval groups

Add approval groups to the PR as needed:

QA checklists

Use the QA checklists below as guides, not rules. Not all checklists will apply to every PR but there could be some overlap.

In all scenarios, changes should be fully tested by the author and verified by the reviewer(s); functionality, responsiveness, etc.

✨ New Component Added
  • The PR has the minor label
  • The component matches the Figma designs.
  • All properties, custom events, and utility functions have e2e and/or unit tests
  • A new Storybook page has been added for the component
  • Tested in all VA breakpoints.
  • Chromatic UI Tests have run and snapshot changes have been accepted by the design reviewer
  • Tested in vets-website using Verdaccio
  • Engineering has approved the PR
  • Design has approved the PR
  • Accessibility has approved the PR
🌱 New Component Variation Added
  • The PR has the minor label
  • The variation matches its Figma design.
  • Any new properties, custom events, or utility functions have e2e and/or unit tests
  • A new story has been added to the component's existing Storybook page
  • Any Chromatic UI snapshot changes have been accepted by a design reviewer
  • Tested in vets-website using Verdaccio
  • Engineering has approved the PR
  • Design has approved the PR
🐞 Component Fix
  • The PR has the patch label
  • Any new properties, custom events, or utility functions have e2e and/or unit tests
  • Any markup changes are evaluated for impact on vets-website.
    • Will any vets-website tests fail from the change?
  • Any Chromatic UI snapshot changes have been reviewed and approved by a designer if necessary
  • Engineering has approved the PR
♿️ Component Fix - Accessibility
  • The PR has the patch label
  • Any new properties, custom events, or utility functions have e2e and/or unit tests
  • Any Chromatic UI snapshot changes have been reviewed and approved by a designer if necessary
  • Engineering has approved the PR
  • Accessibility has approved the PR
🚨 Component Fix - Breaking API Change
  • The PR has the major label
  • vets-website and content-build have been evaluated to determine the impact of the breaking change
  • Any new properties, custom events, or utility functions have e2e and/or unit tests
  • Any Chromatic UI snapshot changes have been reviewed and approved by a designer if necessary
  • Tested in vets-website using Verdaccio
  • Engineering has approved the PR
🔧 Component Update - Non-Breaking API Change
  • The PR has the minor label
  • Any new properties, custom events, or utility functions have e2e and/or unit tests
  • Any Chromatic UI snapshot changes have been reviewed and approved by a designer if necessary
  • Engineering has approved the PR
📖 Storybook Update
  • The PR has the ignore-for-release label
  • Any Chromatic UI snapshot changes have been reviewed and approved by a designer if necessary
  • Engineering has approved the PR
🎨 CSS-Library Update
  • The PR has the css-library label
  • vets-website and content-build have been checked to determine the impact of any breaking changes
  • Engineering has approved the PR

@powellkerry powellkerry requested a review from a team as a code owner March 30, 2026 20:41
@jamigibbs jamigibbs added the ignore-for-release Used if you want to ignore the PR in the generated release notes label Mar 30, 2026
@powellkerry powellkerry merged commit 230e305 into main Mar 30, 2026
9 of 11 checks passed
@powellkerry powellkerry deleted the v56.3.0 branch March 30, 2026 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore-for-release Used if you want to ignore the PR in the generated release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants