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

years for members with multi-year range membership and no other activity are reported incorrectly #750

Closed
3 tasks done
rlskoeser opened this issue Aug 27, 2021 · 1 comment
Assignees
Labels

Comments

@rlskoeser
Copy link
Contributor

rlskoeser commented Aug 27, 2021

Describe the bug
Colum Padric has a 4-year membership documented in the address book with no other known activity.
https://shakespeareandco.princeton.edu/members/colum-padraic/membership/

However, the years displayed on his member page are only the first and last years of his membership, but not the years in the middle when his subscription was still active.

This seems to impact the members search membership years filter as well — searching for Colum and then filtering by an intervening year when his membership was active returned no results.

Expected behavior

  • Membership years should cover all years when a member had an active subscription, even if there is no other activity
  • membership filter should include all years when a member had an active subscription

testing instructions

  • look at Colum Padraic's membership page; confirm membership years are a range instead of start and end years
  • look at James Joyce's membership page; compare with production to confirm membership years are still calculated the same way (i.e. does not include interim years for multiyear borrows)
  • search for Colum Padraic on the members page and confirm you can find him by membership years that fall between his subscription start and end year
@jkotin
Copy link

jkotin commented Dec 18, 2023

This is perfect.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants