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

HYC-1997 - Affiliation Updates #1139

Merged
merged 6 commits into from
Jan 15, 2025
Merged

HYC-1997 - Affiliation Updates #1139

merged 6 commits into from
Jan 15, 2025

Conversation

bbpennel
Copy link
Member

https://unclibrary.atlassian.net/browse/HYC-1997

  • Added and adjusted department vocabulary
    • Fixed incorrect comma delimiters in term fields
  • All solr fields will not store the id for affiliations, both in affiliation_label_tesim and in person objects.
    • Facet filtering by affiliation is by id now, rather than by display form
    • Affiliation values get looked up in the vocab and translated to short_labels before display, in facets and work pages
  • If a department is part of a hierarchy (so it has multiple semi-colon delimited sections in the term field), then the affiliation_label_tesim field will now have an entry for each step of the hierarchy, rather than just the final step before.
    • This means that "School of Medicine" and other top level affiliations will be listed for specific subdepartments, and can be used to find everything by the "School of Medicine".
  • Removed conditional logic to deal person objects not having "index:" fields, since they should all have that now (search prod affirmed this)

…re the id, and then get expanded to short labels for display purposes. Affiliation facet displays short label but searches for id. Work edit form displays short label but displays id for affiliation. Remove split logic for dealing with person objects that don't have index field, since they should all have that now.
@davidcam-src davidcam-src merged commit bc85c6b into main Jan 15, 2025
4 checks passed
@davidcam-src davidcam-src deleted the hyc-1997 branch January 15, 2025 16:16
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