Skip to content

LG-15859 Add service provider name to barcode deadline alert#12051

Merged
shanechesnutt-ft merged 1 commit intomainfrom
sc/LG-15859
Apr 4, 2025
Merged

LG-15859 Add service provider name to barcode deadline alert#12051
shanechesnutt-ft merged 1 commit intomainfrom
sc/LG-15859

Conversation

@shanechesnutt-ft
Copy link
Contributor

@shanechesnutt-ft shanechesnutt-ft commented Apr 3, 2025

🎫 Ticket

Link to the relevant ticket:
LG-15859

🛠 Summary of changes

Add service provider name to the barcode deadline alert for the email and app view.

📜 Testing Plan

Check Emails

Scenario: User creates a pending enrollment

  • Login through the oidc sinatra application selecting the Identity Verified level of service.
  • Create a new account
  • Complete the ID-IPP flow reaching the ready to verify page.
  • Ensure the sentence "Complete this step to access {service provider name}." is present on the page in the blue deadline alert for each supported language
    • English
    • Spanish
    • French
    • Chinese

👀 Screenshots

Ready to Verify Email
Language Screenshot
English Ready_to_verify_email_en
Spanish Ready_to_verify_email_es
French Ready_to_verify_email_fr
Chinese Ready_to_verify_email_zh
Ready to Verify Reminder Email
Language Screenshot
English Ready_to_verify_reminder_email_en
Spanish Ready_to_verify_reminder_email_es
French Ready_to_verify_reminder_email_fr
Chinese Ready_to_verify_reminder_email_zh
Ready to Verify Page
Language Screenshot
English Ready_to_verify_page_en
Spanish Ready_to_verify_page_es
French Ready_to_verify_page_fr
Chinese Ready_to_verify_page_zh

Copy link
Contributor

Choose a reason for hiding this comment

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

Did you get a chance to look at the data in prod and see if any service providers don't have a friendly_name populated? (This isn't a blocking concern, just something I was wondering about after we discussed it.)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have not looked at prod

@shanechesnutt-ft shanechesnutt-ft requested review from a team and kellular and removed request for a team April 4, 2025 15:17
changelog: User-Facing Improvments, In-Person Proofing, Add the service provider name to the barcode page deadline alert
Copy link
Contributor

@eileen-nava eileen-nava left a comment

Choose a reason for hiding this comment

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

Looks good to me. Approved.

@gina-yamada
Copy link
Contributor

gina-yamada commented Apr 4, 2025

✅ Followed testing plan. I can see additional sentence on Ready to Verify Email and Ready to Verify View for IPP, EIPP, and the Reminder for both. (I went through IPP and EIPP flow and checked rails mailers)
✅ Translations added to correct language file. Translations look accurate and punctuations was included as from translation file.
✅ Specs added to reflect change
✅ Additional test locally not coming in from Sinatra- Login.gov is the SP displayed. (I had to type in the URL to move into the flow. A typical user would not move forward.) This should be okay but worth a quick check before merging. You also mentioned @presenter.sp_name is already used in the view so should always be present else I would have suspected this would have been reported broken.

Nice work @shanechesnutt-ft 👏 🎩 🪶 🚀

@gina-yamada gina-yamada self-requested a review April 4, 2025 17:48
Copy link

@kellular kellular left a comment

Choose a reason for hiding this comment

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

This LGTM from UX

@shanechesnutt-ft shanechesnutt-ft merged commit aaf691f into main Apr 4, 2025
1 check passed
@shanechesnutt-ft shanechesnutt-ft deleted the sc/LG-15859 branch April 4, 2025 20:07
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