-
Notifications
You must be signed in to change notification settings - Fork 7
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
Allow restriction to multiple patron locations. (PP-1648) #2049
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2049 +/- ##
==========================================
+ Coverage 90.70% 90.79% +0.08%
==========================================
Files 343 344 +1
Lines 40584 40610 +26
Branches 6588 8793 +2205
==========================================
+ Hits 36813 36872 +59
+ Misses 2501 2484 -17
+ Partials 1270 1254 -16 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The approach here looks good to me!
e66a7f1
to
dbc19e7
Compare
@jonathangreen This could use another look when you have a moment. I'm doing some more manual testing now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
Description
Support more than a single patron location when restricting which patrons can be associated with a given library.
This PR does this by adding an additional Library Identifier Field, "Patron Location", to the fields that can be used to restrict library membership.
Note:
Setting this PR as draft for now, since I still need to remove the recently added single-valued patron location restriction and add/update some more tests, but would be happy to get an initial review on the approach here.
Motivation and Context
Some libraries have multiple locations, so we need a mechanism to support those.
[Jira PP-1648]
How Has This Been Tested?
Checklist