Skip to content
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

LG-15314 | Replace 'partner agency' with SP name #11686

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

n1zyy
Copy link
Member

@n1zyy n1zyy commented Dec 23, 2024

🎫 Ticket

Link to the relevant ticket:
LG-15314

🛠 Summary of changes

Due to a misunderstanding with the Department of State, the strings for a prompt redirecting users to their service provider when they hadn't connected their account to them used "partner agency" as a literal string rather than an interpolation.

This fixes that, and also omits the message in the case that the name is nil since it is otherwise incomprehensible and contains no link.

That required some test changes, and also led to a bit of refactoring. Shoutout to @jmhooper for encouraging that part.

👀 Screenshots

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

Before:

Screenshot 2024-12-20 at 2 38 10 PM

After:

No SP name, so CTA not displayed:
(Otherwise it would read "Connect your account to. Sign in to to access services.")
Screenshot 2024-12-20 at 5 10 03 PM

English:
Screenshot 2024-12-20 at 5 14 02 PM

Spanish:
Screenshot 2024-12-20 at 5 14 21 PM

French:
Screenshot 2024-12-20 at 5 14 15 PM

Chinese (simplified):
Screenshot 2024-12-20 at 5 14 09 PM

Due to a misunderstanding with DOS, the translations had the
literal string "service provider". Fixed!

Conditionally hides the display if there is no SP (which shouldn't
happen in prod) because otherwise the message is nonsense and has
no clickable link text.

changelog: User-Facing Improvements, Account screen, Show SP name in return-to-SP CTA
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