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

Email Error #808

Merged
merged 3 commits into from
Dec 19, 2023
Merged

Email Error #808

merged 3 commits into from
Dec 19, 2023

Conversation

CalebPena
Copy link
Collaborator

What (if any) features are you implementing?

  • Add an error message if the server throws an error for email address.

Were there any issues that arose?

  • We can not currently determine the exact error with the email, so we have to use a generic error message. @sydney-devine Let me know if you would like to change any of the wording on the error message.

image

@CalebPena CalebPena added the impact-medium Moderate possible impact on data, security or user experience. Req 2 reviews. label Dec 12, 2023
@sydney-devine
Copy link
Collaborator

@CalebPena

Some typos and updated language.

Please enter a valid email address. This error could also be caused by entering an email address that is already in our system. If the error persists, remember that this question is optional and will not impact your MyFriendBen results. You can skip this question by deselecting the boxes at the top of the page and pressing continue.

Can we align the error icon with the first line?

@CalebPena CalebPena merged commit 243c7f9 into main Dec 19, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact-medium Moderate possible impact on data, security or user experience. Req 2 reviews.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants