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

Many notifications when trying to create an Eperson with an already existing email #1325

Closed
corrad82-4s opened this issue Sep 23, 2021 · 1 comment · Fixed by #1336
Closed
Assignees
Labels
bug component: administrative tools Related to the admin menu or tools e/6 Estimate in hours
Milestone

Comments

@corrad82-4s
Copy link
Contributor

Describe the bug
When trying to register an Eperson with an already existing email, the save action does not create a new Eperson, but the form submission is performed and user sees three different error notifications.

image

To Reproduce
Steps to reproduce the behavior:

  1. Login as administrator on dspace7 demo
  2. go to Access Control -> People
  3. click "add Eperson"
  4. fill the form with an email of an already existing user

Expected behavior
Three different notifications might be a bit confusing for the user, we believe he / she should be warned about the already existing email, but with a single error message, or even better, made unable to perform the "save" action if the email is invalid or already in use.

We (4 Science) are available to take in charge this improvement by adding a validator around the eperson creation form that highlights the email input as invalid and preventing form submit action. To add this feature and its test we estimate 4 to 8 hours of effort.

@corrad82-4s corrad82-4s added bug needs triage New issue needs triage and/or scheduling labels Sep 23, 2021
@tdonohue
Copy link
Member

@corrad82-4s : I agree this behavior is not great. The estimate sounds reasonable, so I'll assign this back to you for you/your team to work on.

@tdonohue tdonohue added component: administrative tools Related to the admin menu or tools e/6 Estimate in hours and removed needs triage New issue needs triage and/or scheduling labels Sep 23, 2021
@tdonohue tdonohue added this to To Do in DSpace 7.1 release via automation Sep 23, 2021
@tdonohue tdonohue added this to the 7.1 milestone Sep 23, 2021
@tdonohue tdonohue moved this from To Do to In Progress in DSpace 7.1 release Oct 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug component: administrative tools Related to the admin menu or tools e/6 Estimate in hours
Projects
No open projects
DSpace 7.1 release
  
In Progress
Development

Successfully merging a pull request may close this issue.

2 participants