Skip to content

LG-11363 Update content to emphasize visiting the PO in IPP#9494

Merged
gina-yamada merged 15 commits intomainfrom
yamada/LG-11363-UpdateContentForPoSearchInIPP
Nov 3, 2023
Merged

LG-11363 Update content to emphasize visiting the PO in IPP#9494
gina-yamada merged 15 commits intomainfrom
yamada/LG-11363-UpdateContentForPoSearchInIPP

Conversation

@gina-yamada
Copy link
Contributor

@gina-yamada gina-yamada commented Oct 31, 2023

🎫 Ticket

LG-11363 Update content to emphasize visiting the PO in IPP

Translation can be found here for quick reference

🛠 Summary of changes

  • Update content & translations on /verify/document_capture view
  • Update content & translations on /verify/doc_auth/document_capture#prepare

📜 Testing Plan

  • Step 1 Create an account and start to identity your identity, upload images so that you fail proofing
  • Step 2 Inspect content at url /verify/document_capture to confirm all updates are made
  • Step 3 Click Try at a Post Office button
  • Step 4 Inspect content at url /verify/document_capture#prepare to confirm all updates are made
  • Step 5 Repeat steps 2-4 for Spanish and French translations
  • Step 6 Confirm visually other views are not affected by changes [or if access to repostory, that no other instances of variables change are being used with the exception of tests so show that content on other views is not affected]

👀 Screenshots for verify/document_capture

English Screenshot 2023-10-31 at 3 07 48 PM
Spanish
  • updated screenshot of Spanish in comments
Screenshot 2023-10-31 at 3 09 40 PM
French Screenshot 2023-10-31 at 3 10 22 PM

👀 Screenshots for verify/document_capture#prepare

English Screenshot 2023-10-31 at 4 14 10 PM
Spanish
  • updated screenshot of Spanish in comments
Screenshot 2023-10-31 at 4 14 38 PM
French Screenshot 2023-10-31 at 4 14 56 PM

✅ List of Changes

verify/document_capture

  • We'll update the h2 to: "Try verifying your identity at a Post Office"
  • We'll update the description under the h2 to: You'll start this process by entering your information online, and continue to verify your identity in person at a participating Post Office near you."
  • We'll change the CTA to: "Try at a Post Office"

verify/document_capture#prepare

  • We'll update the h2 to: "Verify your identity at a Post Office"
  • We’ll update the body to: “Complete the following steps to verify your identity at a Post Office:” Gina added this item. It is in the attached image. I think it was just forgotten on this list because the translations have also been updated to match the image in the issue.
  • We'll update step 3 to: "Enter your primary phone number or the number that you use most often. If we’re able to verify your information, you'll receive a barcode."
  • We'll add a new step 4: "Bring the barcode and your state-issued ID to a participating Post Office."
  • And we'll add a paragraph after the steps: "You’ll receive an email with the results within 24 hours of your visit to the Post Office. If your identity verification was successful, you’ll be able to sign in and share your verified information with your agency."

@gina-yamada gina-yamada marked this pull request as ready for review November 1, 2023 14:39
@gina-yamada
Copy link
Contributor Author

gina-yamada commented Nov 1, 2023

Updated Spanish translation of Post Office to be lower case after meeting with @carmenrosalop

Screenshot 2023-11-01 at 12 36 11 PM Screenshot 2023-11-01 at 12 36 26 PM

location: Find a participating Post Office
prepare: Verify your identity in person
prepare: Verify your identity at a Post Office
state_id_milestone_2: Enter the information on your state‑issued ID
Copy link
Contributor Author

@gina-yamada gina-yamada Nov 2, 2023

Choose a reason for hiding this comment

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

There are 2 instance of in_person_proofing.headings.prepare in the repo that are not in spec files. The first is the title on the page that was on the list of changes. The other instance is changing the tab title. See screenshot.

Screenshot 2023-11-02 at 10 15 55 AM

Copy link

@carmenrosalop carmenrosalop left a comment

Choose a reason for hiding this comment

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

LGTM!!!

@gina-yamada gina-yamada requested review from a team and allthesignals November 2, 2023 17:46
expect(getByText('in_person_proofing.body.prepare.verify_step_visit_post_office')).to.exist();
});

it('renders about and additional information', () => {
Copy link
Contributor

@svalexander svalexander Nov 2, 2023

Choose a reason for hiding this comment

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

small nit - can we update the language to ...information steps

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes! Resolved with commit 6156de7

Copy link
Contributor

@svalexander svalexander left a comment

Choose a reason for hiding this comment

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

LGTM

@gina-yamada gina-yamada merged commit 3afe340 into main Nov 3, 2023
@gina-yamada gina-yamada deleted the yamada/LG-11363-UpdateContentForPoSearchInIPP branch November 3, 2023 11:34
@matthinz matthinz mentioned this pull request Nov 6, 2023
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.

3 participants