Skip to content

LG-8930: Show formatted date of birth on in-person proofing verify step#7975

Merged
tomas-nava merged 3 commits intomainfrom
tomas/lg-8930-full-month-for-dob
Mar 14, 2023
Merged

LG-8930: Show formatted date of birth on in-person proofing verify step#7975
tomas-nava merged 3 commits intomainfrom
tomas/lg-8930-full-month-for-dob

Conversation

@tomas-nava
Copy link
Contributor

🎫 Ticket

LG-8930

🛠 Summary of changes

On the Verify your information screen, show the Date of Birth in full month format.

📜 Testing Plan

  • Create an account
  • Go through the in-person proofing process
  • Note date of birth format on the Verify your information screen

👀 Screenshots

before (en): en-before
after (en, es, fr): en es fr

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! One naming suggestion.

Tomas Apodaca added 2 commits March 14, 2023 12:55
changelog: User-Facing Improvements, Verify your information for in person proofing, Show formatted date of birth
@mitchellhenke mitchellhenke force-pushed the tomas/lg-8930-full-month-for-dob branch from 7e8b182 to 6fb3a1d Compare March 14, 2023 17:56
@carmenrosalop
Copy link

Looks good! Thanks Tomas

expect(page).to have_text(InPersonHelper::GOOD_FIRST_NAME)
expect(page).to have_text(InPersonHelper::GOOD_LAST_NAME)
expect(page).to have_text(InPersonHelper::GOOD_DOB)
expect(page).to have_text(InPersonHelper::GOOD_DOB_FORMATTED_EVENT)

Choose a reason for hiding this comment

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

The changes look good in the three languages

@tomas-nava tomas-nava merged commit 7517dc9 into main Mar 14, 2023
@tomas-nava tomas-nava deleted the tomas/lg-8930-full-month-for-dob branch March 14, 2023 22:11
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.

4 participants