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

The 'error' message to check the new email address is alarming #1239

Open
6 tasks
adriannelee opened this issue Jun 1, 2023 · 10 comments
Open
6 tasks

The 'error' message to check the new email address is alarming #1239

adriannelee opened this issue Jun 1, 2023 · 10 comments
Assignees
Labels
Content l Contenu Tasks to write, edit or delete content from any Notify pages Low Priority | Faible priorité UX Design and prototyping

Comments

@adriannelee
Copy link
Collaborator

Description

As a Sender, I need to be able to check how Notify formatted the Sender email address that I inputted without being alarmed

WHY are we building?
Currently, when you try to create a Sender email address with bad formatting, Notify will correct the formatting and prompt you to check. However, the prompt is in red and looks like an error.

WHAT are we building?
A less alarming 'error' message

VALUE created by our solution
Better UX

Documentation and Artifacts

Screen Shot 2023-06-01 at 5.39.22 PM.png
Good docs, figma mockups, ADRs, screenshots etc.

Acceptance Criteria

Given some context, when (X) action occurs, then (Y) outcome is achieved

  • When a new Sender is creating a new service and they input the Sender email address with bad formatting, they're not alarmed by the 'check' message
  • Generate appropriate log messages so that executions of this feature can be tracked
  • Can misuse of this feature cause harm? If yes, create an alert
  • Update the status of related findings, insights, and hypotheses on the Research Airtable
  • Once change/fix/feature is implemented, link relevant Airtable records to design artifacts (Figma)
  • A11y
  • Bilingualism
  • Privacy considerations
  • Security controls in place
  • Measuring success and metrics

Related Research Airtable records

QA Steps

  • Tested in a realistic production scenario
@adriannelee adriannelee added Content l Contenu Tasks to write, edit or delete content from any Notify pages Low Priority | Faible priorité UX Design and prototyping labels Jun 1, 2023
@adriannelee adriannelee removed this from A11Y work Jun 8, 2023
@yaelberger-commits
Copy link
Collaborator

Hey team! Please add your planning poker estimate with Zenhub @amazingphilippe @YedidaZalik

@YedidaZalik
Copy link

Starting working on this

One of the main concerns was the red. I started trying out the flow and the error message I get is not in red, so maybe we already fixed this?
(I would still suggest changing "Make sure" to "Check" which is actually the word we use in Figma)Screenshot 2024-12-24 at 2.43.33 PM.png

@yaelberger-commits
Copy link
Collaborator

@YedidaZalik did some validation testing and found the red box is no longer appearing. We should ask @andrewleith to check and make sure it's been changed. Also we should use "Check" instead of "Make sure" as per Yedida's suggestion and that's what's in the Figma mockups.

@smcmurtry
Copy link
Collaborator

smcmurtry commented Dec 30, 2024

I took a look at this since Andrew is on vacation.

This change to remove the red error message and to use the grey box was made 8 months ago in this PR and this card.

The page now uses the "confirmation" design pattern (grey box) instead of the error pattern.

Screenshot 2024-12-30 at 2.18.08 PM.png

I can change the copy to "Check that your email address is formatted correctly" but I don's see a french translation for this in Figma.

@YedidaZalik
Copy link

YedidaZalik commented Dec 30, 2024

@smcmurtry
Copy link
Collaborator

@yaelberger-commits
Copy link
Collaborator

Once merged to Staging we'll do QA

@smcmurtry
Copy link
Collaborator

merged and ready for QA

@YedidaZalik
Copy link

QA'd in staging - LGTM!

@YedidaZalik
Copy link

QA'd in staging - LGTM in both languages!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content l Contenu Tasks to write, edit or delete content from any Notify pages Low Priority | Faible priorité UX Design and prototyping
Projects
None yet
Development

No branches or pull requests

5 participants