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

Removing an e-mail in the user does not check in all organisations if there is still an e-mail communication channel #2281

Closed
pronguen opened this issue Aug 9, 2021 · 0 comments · Fixed by #2889
Assignees
Labels
bug Breaks something but is not blocking f: user management p-Medium Medium priority (to be solved within 1-2 years)

Comments

@pronguen
Copy link
Contributor

pronguen commented Aug 9, 2021

Describe the bug

Removing an e-mail in the user does not check in all organisations if there is still an e-mail communication channel.

See linked issue: #1580

To Reproduce

  1. Find a patron with e-mail, registered in 2 organisations, both with e-mail communication channel
  2. From one organisation, set the communication channel to mail
  3. ...and then remove the e-mail in the user
  4. See that it is possible. There is an inconsistency in the other organisation.

Expected behavior

Removing an e-mail in the user should check in all organisations if there is still an e-mail communication channel.

Context

  • server: ils.test.rero.ch
  • version: v1.4.6
@pronguen pronguen added f: user management bug Breaks something but is not blocking p-Medium Medium priority (to be solved within 1-2 years) labels Aug 9, 2021
BadrAly pushed a commit to BadrAly/rero-ils that referenced this issue May 9, 2022
No reset password notifications are sent after any changes made
to the patron/user record for both admin and public interfaces.

The communication channel is updated automatically after updates
made to the patron profile by the patron himself.

* Fixes rero#2313
* Fixes rero#2281
* Fixes rero#1580

Co-Authored-by: Aly Badr <[email protected]>
BadrAly pushed a commit to BadrAly/rero-ils that referenced this issue May 9, 2022
No reset password notifications are sent after any changes made
to the patron/user record for both admin and public interfaces.

The communication channel is updated automatically after updates
made to the patron profile by the patron himself.

* Fixes rero#2313
* Fixes rero#2281
* Fixes rero#1580
* Fixes rero#2887

Co-Authored-by: Aly Badr <[email protected]>
BadrAly pushed a commit to BadrAly/rero-ils that referenced this issue May 9, 2022
No reset password notifications are sent after any changes made
to the patron/user record for both admin and public interfaces.

The communication channel is updated automatically after updates
made to the patron profile by the patron himself.

* Fixes rero#2313
* Fixes rero#2281
* Fixes rero#1580
* Fixes rero#2887

Co-Authored-by: Aly Badr <[email protected]>
BadrAly pushed a commit to BadrAly/rero-ils that referenced this issue May 10, 2022
No reset password notifications are sent after any changes made
to the patron/user record for both admin and public interfaces.

The communication channel is updated automatically after updates
made to the patron profile by the patron himself.

* Fixes rero#2313
* Fixes rero#2281
* Fixes rero#1580
* Fixes rero#2887

Co-Authored-by: Aly Badr <[email protected]>
BadrAly pushed a commit to BadrAly/rero-ils that referenced this issue May 10, 2022
No reset password notifications are sent after any changes made
to the patron/user record for both admin and public interfaces.

The communication channel is updated automatically after updates
made to the patron profile by the patron himself.

* Fixes rero#2313
* Fixes rero#2281
* Fixes rero#1580
* Fixes rero#2887

Co-Authored-by: Aly Badr <[email protected]>
BadrAly pushed a commit to BadrAly/rero-ils that referenced this issue May 10, 2022
No reset password notifications are sent after any changes made
to the patron/user record for both admin and public interfaces.

The communication channel is updated automatically after updates
made to the patron profile by the patron himself.

* Fixes rero#2313
* Fixes rero#2281
* Fixes rero#1580
* Fixes rero#2887

Co-Authored-by: Aly Badr <[email protected]>
BadrAly pushed a commit to BadrAly/rero-ils that referenced this issue May 10, 2022
No reset password notifications are sent after any changes made
to the patron/user record for both admin and public interfaces.

The communication channel is updated automatically after updates
made to the patron profile by the patron himself.

* Fixes rero#2313
* Fixes rero#2281
* Fixes rero#1580
* Fixes rero#2887

Co-Authored-by: Aly Badr <[email protected]>
BadrAly pushed a commit that referenced this issue May 10, 2022
No reset password notifications are sent after any changes made
to the patron/user record for both admin and public interfaces.

The communication channel is updated automatically after updates
made to the patron profile by the patron himself.

* Fixes #2313
* Fixes #2281
* Fixes #1580
* Fixes #2887

Co-Authored-by: Aly Badr <[email protected]>
BadrAly pushed a commit that referenced this issue May 12, 2022
No reset password notifications are sent after any changes made
to the patron/user record for both admin and public interfaces.

The communication channel is updated automatically after updates
made to the patron profile by the patron himself.

* Fixes #2313
* Fixes #2281
* Fixes #1580
* Fixes #2887

Co-Authored-by: Aly Badr <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Breaks something but is not blocking f: user management p-Medium Medium priority (to be solved within 1-2 years)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants