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

suggest local library for ILL ineligible #614

Merged
merged 2 commits into from
Apr 18, 2023
Merged

suggest local library for ILL ineligible #614

merged 2 commits into from
Apr 18, 2023

Conversation

Smullz622
Copy link
Contributor

@Smullz622 Smullz622 commented Apr 12, 2023

Fixes #578

I spoke with Ruth & we're extending this to any patrons who are not eligible for ILL, not just PA residents.
Patrons who are not ILL eligible will see the following message:

@Smullz622 Smullz622 requested review from banukutlu and ajkiessl April 12, 2023 19:05
@Smullz622 Smullz622 changed the title 578 local library suggest local library for ILL ineligible Apr 12, 2023
@banukutlu banukutlu added this to the 1.2.x milestone Apr 12, 2023
Copy link
Contributor

@ajkiessl ajkiessl left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@banukutlu banukutlu left a comment

Choose a reason for hiding this comment

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

LGTM!

@banukutlu banukutlu merged commit 5a5e8be into main Apr 18, 2023
@banukutlu banukutlu deleted the 578-local-library branch April 18, 2023 15:28
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.

No Recalls: Suggest PA residents try their local libraries
3 participants