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

Us 2034 multiple patron #1858

Merged
merged 1 commit into from
Apr 30, 2021
Merged

Us 2034 multiple patron #1858

merged 1 commit into from
Apr 30, 2021

Conversation

jma
Copy link
Contributor

@jma jma commented Apr 29, 2021

Why are you opening this PR?

  • Which task/US does it implement?
  • Which issue does it fix?

Dependencies

My PR depends on the following rero-ils-ui"s PR(s):

  • rero/rero-ils-ui#

How to test?

  • What command should I have to run to test your PR?
  • What should I test through the UI?

Code review check list

  • Commit message template compliance.
  • Commit message without typos.
  • File names.
  • Functions names.
  • Functions docstrings.
  • Unnecessary commited files?
  • Cypress tests successful?

* Changed logged user structure.
* Updated users import cli to accept multiple patron on same user.
* Replaces the string `current_patron` by `current_librarian` in the
  entire project.
* Adds a new `current_patrons` containing the list of the logged user
  patron accounts.
* Removes useless jinja detailed view configuration and templates.
* Fixes link redirection for the ill request.
* Cleans the patron request Web API.
* Fixes tests and permissions.
* Replaces `current_organisation.pid` by `current_librarian.organsiation_pid.
* Fixes tests and premissions.
* Gets the patron request rank given a patron instead of a barcode.
* Fixes translated string on circulation.
* Hides the profile menu given the right roles.
* Adds organisations information in the User API.
* Closes #1837.
* Closes #1460.
* Closes #1851.
* Closes #1804.
* Closes #1634.
* Closes #1490.

Co-Authored-by: Bertrand Zuchuat <[email protected]>
Co-Authored-by: Johnny Mariéthoz <[email protected]>
@jma jma force-pushed the US-2034-multiple-patron branch from fd40ee9 to e2797e4 Compare April 29, 2021 17:14
@jma jma merged commit 714fed6 into dev Apr 30, 2021
@jma jma deleted the US-2034-multiple-patron branch June 1, 2021 07:55
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

Successfully merging this pull request may close these issues.

2 participants