Skip to content

LG-11383 Update Verify Info Screen#9556

Merged
theabrad merged 9 commits intomainfrom
abrad-lg-11383-update-verify-info
Nov 8, 2023
Merged

LG-11383 Update Verify Info Screen#9556
theabrad merged 9 commits intomainfrom
abrad-lg-11383-update-verify-info

Conversation

@theabrad
Copy link
Contributor

@theabrad theabrad commented Nov 7, 2023

🎫 Ticket

LG-11383

🛠 Summary of changes

We are updating the Verify Info screen to clarify the purpose of the screen.

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Go through IdV
  • Reach the Verify Info screen
  • Confirm that there is text under the header & the call to action button says "submit" instead of "continue"

👀 Screenshots

If relevant, include a screenshot or screen capture of the changes.

Before: Screenshot 2023-11-07 at 12 00 14 PM
After: Screenshot 2023-11-07 at 11 49 47 AM Screenshot 2023-11-07 at 11 49 59 AM Screenshot 2023-11-07 at 11 50 10 AM

changelog: User-Facing Improvements, IdV Verify Info, add text
clarifying purpose of verify info screen
@theabrad theabrad requested a review from a team November 7, 2023 17:04
},
},
).with_content(t('forms.buttons.continue')) %>
).with_content(t('forms.buttons.submit.default')) %>
Copy link
Contributor

Choose a reason for hiding this comment

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

The ticket doesn't say to change the In person verify info submit button as far as I saw. Did we decide to keep just the buttons in sync?

Copy link
Contributor

@soniaconnolly soniaconnolly left a comment

Choose a reason for hiding this comment

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

LGTM, since @artfulaction confirmed we want to change button labels on both VerifyInfo screens.

@theabrad theabrad merged commit e54b04e into main Nov 8, 2023
@theabrad theabrad deleted the abrad-lg-11383-update-verify-info branch November 8, 2023 15:19
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