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.

Sign up for GitHub

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

Jump to bottom

Creating a user with a patron role and with an existing RERO ID email does not works. #1454

Closed
jma opened this issue Nov 18, 2020 · 0 comments · Fixed by #1463
Closed

Creating a user with a patron role and with an existing RERO ID email does not works. #1454

jma opened this issue Nov 18, 2020 · 0 comments · Fixed by #1463
Assignees
@jma
Labels
bug Breaks something but is not blocking f: user management p-Low Low priority
Milestone
v0.14.1

Comments

@jma
Copy link
Contributor

jma commented Nov 18, 2020 •
edited by iGormilhit
Loading

Describe the bug
Creating a user with a patron role and an already existing email cause the spinner to run forever.

To Reproduce
Steps to reproduce the behavior:

  1. Register a new RERO ID account.
  2. Login with a librarian.
  3. Create a new user with the patron role and paste the email of the previously created RERO ID.
  4. The form is not submitted.
@jma jma added triage bug Breaks something but is not blocking labels Nov 18, 2020
@jma jma added this to the v0.14.1 milestone Nov 18, 2020
@jma jma self-assigned this Nov 18, 2020
@iGormilhit iGormilhit added p-Low Low priority f: user management and removed triage labels Nov 19, 2020
jma added a commit to jma/rero-ils that referenced this issue Nov 19, 2020
@jma
patrons: fix patron creation and modification
377dd44 
* Fixes a patron creation with an existing user account, closes: rero#1454.
* Fixes user profile information modification for a user without email,
  closes rero#1459.
* Fixes a required email for a patron creation if the communication
  channel is email, closes rero#1455.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
@jma jma mentioned this issue Nov 19, 2020
Merged
6 tasks
jma added a commit to jma/rero-ils that referenced this issue Nov 19, 2020
@jma
patrons: fix patron creation and modification
7b091c9 
* Fixes a patron creation with an existing user account, closes: rero#1454.
* Fixes user profile information modification for a user without email,
  closes rero#1459.
* Fixes a required email for a patron creation if the communication
  channel is email, closes rero#1455.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
jma added a commit to jma/rero-ils that referenced this issue Nov 19, 2020
@jma
patrons: fix patron creation and modification
1f212b4 
* Fixes a patron creation with an existing user account, closes: rero#1454.
* Fixes user profile email change does not update the patron record,
  closes rero#1458.
* Fixes a required email for a patron creation if the communication
  channel is email, closes rero#1455.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
jma added a commit to jma/rero-ils that referenced this issue Nov 19, 2020
@jma
patrons: fix patron creation and modification
03e8d83 
* Fixes a patron creation with an existing user account, closes: rero#1454.
* Fixes user profile email change does not update the patron record,
  closes rero#1458.
* Fixes a required email for a patron creation if the communication
  channel is email, closes rero#1455.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
jma added a commit to jma/rero-ils that referenced this issue Nov 19, 2020
@jma
patrons: fix patron creation and modification
90d22c2 
* Fixes patron creation with an existing user account, closes rero#1454.
* Ensures that when the patron updates his/her email in the RERO ID, the
  modification is synched to the patron record, closes rero#1458.
* Makes the email required if the communication channel is email, closes rero#1455.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
@iGormilhit iGormilhit changed the title Create a user with a patron role with an existing RERO ID email does not works. Creating a user with a patron role and with an existing RERO ID email does not works. Nov 19, 2020
@jma jma closed this as completed in #1463 Nov 19, 2020
jma added a commit that referenced this issue Nov 19, 2020
@jma
patrons: fix patron creation and modification
747b790 
* Fixes patron creation with an existing user account, closes #1454.
* Ensures that when the patron updates his/her email in the RERO ID, the
  modification is synched to the patron record, closes #1458.
* Makes the email required if the communication channel is email, closes #1455.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
rerowep pushed a commit to rerowep/rero-ils that referenced this issue Nov 20, 2020
@jma @rerowep
patrons: fix patron creation and modification
b94399e 
* Fixes patron creation with an existing user account, closes rero#1454.
* Ensures that when the patron updates his/her email in the RERO ID, the
  modification is synched to the patron record, closes rero#1458.
* Makes the email required if the communication channel is email, closes rero#1455.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jma jma

Labels
bug Breaks something but is not blocking f: user management p-Low Low priority
Projects
None yet
Milestone
v0.14.1
Development

Successfully merging a pull request may close this issue.

patrons: fix patron creation and modification jma/rero-ils
2 participants
@jma @iGormilhit