-
Notifications
You must be signed in to change notification settings - Fork 71
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
VACMS-20435: Updates description meta tag for va forms to use field_va_form_name #20697
Conversation
Checking composer.lock changes... |
Cypress Accessibility Violations
|
@Shamiaalam This will need your review once @omahane is done with his PR review since it has UX changes. Let me know once you area ready to reivew and we can sync up if needed! |
Checking composer.lock changes... |
Scratch that, I thought I was commenting on a different PR. Ignore this! |
Description
Updates description meta tag for va forms to use field_va_form_name
Relates to #20435
Testing done
Manual
but we need a full TB instance to check end-to-end that the meta tag is making its way to the FE.confirmed.Screenshots
QA steps
What needs to be checked to prove this works?
What needs to be checked to prove it didn't break any related things?
What variations of circumstances (users, actions, values) need to be checked?
As user uid with user_role
Definition of Done
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?