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

permissions: fix patron read permission #3281

Merged
merged 1 commit into from
Mar 14, 2023
Merged

Conversation

jma
Copy link
Contributor

@jma jma commented Mar 14, 2023

  • Fixes unable to read a patron record for a librarian with a patron role.

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?

* Fixes unable to read a patron record for a librarian with a patron
  role.

Co-authored-by: Johnny Mariéthoz <[email protected]>
Co-authored-by: Renaud Michotte <[email protected]>
@jma jma requested a review from zannkukai March 14, 2023 09:23
@github-actions github-actions bot added the f: permissions Concerns the rights management label Mar 14, 2023
Copy link
Contributor

@zannkukai zannkukai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thx for the fix !

@jma jma marked this pull request as ready for review March 14, 2023 12:52
@jma jma merged commit 6080c01 into rero:staging Mar 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
f: permissions Concerns the rights management
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants