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

Make "Email already in use" error more explanatory. #1624

Closed
Benunc opened this issue Apr 17, 2017 · 1 comment
Closed

Make "Email already in use" error more explanatory. #1624

Benunc opened this issue Apr 17, 2017 · 1 comment
Assignees
Milestone

Comments

@Benunc
Copy link
Member

Benunc commented Apr 17, 2017

Issue Overview

Give checks to see if a user who is logged in is trying to make a donation using an email that already exists in the Give donors database. If it does, it outputs this error:

screen shot 2017-04-17 at 2 24 02 pm

That error message is not clear as to why exactly that's a problem. I've been confused by it, and today got a support ticket from a confused customer.

Expected Behavior

The error message should say something like

You are logged in as admin@example.com, and are submitting a donation as leroy@jenkins.com, which is an existing Give Donor. To ensure that the email address is tied to the correct donor, please submit this donation from a logged-out browser, or choose another email address.

Or, even better, Give should be able to tie the donation made to that already existing donor, as it does for a logged out donor. That's outside the scope of this issue, though I could write a separate one if that seems appropriate.

Current Behavior

The error message is confusing.

Possible Solution

Update this line with a more instructive error: https://github.com/WordImpress/Give/blob/3c0417945a510b5cf249ef17a60ed96daf9c275d/includes/process-donation.php#L191

Steps to Reproduce (for bugs)

  1. On a clean install, submit a donation as leroy@jenkins.com
  2. Then log into an account other than the one associated with leroy@jenkins.com
  3. Submit a new donation while logged in as the other user
  4. You'll get the error message above.
@mathetos
Copy link
Member

Great ticket @Benunc Totally agree.

@DevinWalker DevinWalker self-assigned this Apr 28, 2017
@DevinWalker DevinWalker added this to the 1.8.7 milestone Apr 28, 2017
DevinWalker added a commit that referenced this issue Apr 28, 2017
@DevinWalker DevinWalker mentioned this issue Apr 28, 2017
3 tasks
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

No branches or pull requests

3 participants