18F / identity-idp

Login.gov Core App: Identity Provider (IdP)
https://secure.login.gov/
Other
524 stars 112 forks source link

LG-14587 | Updates to 'You successfully verified your identity' email #11295

Closed n1zyy closed 1 month ago

n1zyy commented 1 month ago

🎫 Ticket

Link to the relevant ticket: LG-14587

🛠 Summary of changes

[!IMPORTANT] There is a desire from leadership/product to have this story go out in the next deploy (October 1). If reviewers have suggested changes, I'd be grateful if you can be clear which are blocking and which can be resolved in a follow-up story if need be. (I'll try to get them all fixed ahead of time, though!)

In this story, we make some changes to the account_verified email. Namely:

👀 Screenshots

You can preview the message at https://review-mattw-lg-1-jygxb8.reviewapps.identitysandbox.gov/rails/mailers/user_mailer/account_verified

Please note that the screenshots below were taken before this change: https://github.com/18F/identity-idp/pull/11295/commits/db541009710b51f245edbff6447a693b38298c33

With Service Provider with `return_to_sp_url` specified (Note that `localhost:9292` is the Service Provider's URL, Example Sinatra App here.) ![Screenshot 2024-09-30 at 2 25 01 PM](https://github.com/user-attachments/assets/b070de37-4ee4-4022-bb29-7b7aa4775636) ![Screenshot 2024-09-30 at 2 25 29 PM](https://github.com/user-attachments/assets/05d07fc7-f5ab-4c62-9b4b-0c958a698ab2) ![Screenshot 2024-09-30 at 2 25 25 PM](https://github.com/user-attachments/assets/26783cc3-507e-4a7b-95ba-add37b2194a8) ![Screenshot 2024-09-30 at 2 25 20 PM](https://github.com/user-attachments/assets/b16d06d7-2a6b-4769-8c87-9a87771ddd0d)
With Service Provider with NO return_to_sp_url (I set the fictional SP name to "The FBI" because it made for a cool demo. Because there is no `return_to_sp_url`, the user is just told to sign into the SP but there's nothing we can link to, so we omit the link/button.) ![Screenshot 2024-09-30 at 2 30 01 PM](https://github.com/user-attachments/assets/21e0f415-0daa-4df6-b56a-c52a51d09621) ![Screenshot 2024-09-30 at 2 29 57 PM](https://github.com/user-attachments/assets/3fe3ddef-f408-43de-8365-4cc01b387d1c) ![Screenshot 2024-09-30 at 2 29 53 PM](https://github.com/user-attachments/assets/7ebd7858-62fb-4e5d-b3ca-515d47b9ec0b) ![Screenshot 2024-09-30 at 2 29 49 PM](https://github.com/user-attachments/assets/698313ec-9dbb-480f-b90e-69e80b4344d2)
With NO service provider (Note that `localhost:3000` is equivalent to Login.gov in prod: i.e., with no SP, the user is prompted to sign into Login.gov directly.) ![Screenshot 2024-09-30 at 2 32 23 PM](https://github.com/user-attachments/assets/fce6ebcc-667b-48a7-9b24-5f3110b55e34) ![Screenshot 2024-09-30 at 2 32 19 PM](https://github.com/user-attachments/assets/3c16a6a5-d0d2-4c21-9cbb-a2a5d9a69542) ![Screenshot 2024-09-30 at 2 32 15 PM](https://github.com/user-attachments/assets/fb7383b4-37ef-4119-b490-b6dcd349d6ed) ![Screenshot 2024-09-30 at 2 32 11 PM](https://github.com/user-attachments/assets/19740c7c-bae0-4b1a-bb3d-41395189cc01)
n1zyy commented 1 month ago

There is one failing test:

  1) I18n config/locales/es.yml does not contain any translations that hardcode APP_NAME
     Failure/Error: expect(bad_keys).to be_empty
       expected `{"user_mailer.account_verified.contact_link"=>"contacte con el servicio de asistencia de Login.gov"}.empty?` to be truthy, got false

I am going to deal with this Monday morning. The problem is that this string is "contact us" in other languages, but they made the Spanish one include the string "Login.gov" so now the linter is angry. 🙀

matthinz commented 1 month ago

The email looked good in all the clients I was able to test in. Maybe slightly wonky in macOS mail, but what isn't? That is, this is not something we're going to fix in this PR!

macOS mail macOS mail rendering
Gmail (iOS) ![image](https://github.com/user-attachments/assets/51cb76f7-6de1-493b-9cf2-a2a14c53f65f)