-
Notifications
You must be signed in to change notification settings - Fork 1
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
Mapping subjects #23
Comments
|
|
|
|
|
In the wiki, subject fields table does not include Primary subjects: Also I think we should try to keep these two fields inline with subject display. Could you pls review the fields mappings again? |
@ruthtillman could you also review the subject facet mapping again? https://github.com/psu-libraries/psulib_blacklight/wiki/Subject-Fields-Index-and-Display#subject-facet Are subfields correct here: |
@banukutlu flagging to review first thing tomorrow morning. Of your question and Charlie's today, his was faster to answer. |
@banukutlu one quick answer, the repeated a is correct. It's a little weird but occasionally the data may have that. It won't repeat if more than one a doesn't exist. |
@banukutlu updated wiki. Added fields requested, removed 655 from index for this, also removed ATOU because I didn't like a couple things it would be indexing so made more specific. |
@ruthtillman an example with the latest mappings:
https://blackcat01qa.libraries.psu.edu/catalog/21601671/librarian_view
|
@ruthtillman can you clarify the below paragraph from our subjects doc:
|
@banukutlu I tried clarifying -- does that help? |
yes, Thank you! |
https://github.com/psu-libraries/psulib_blacklight/wiki/Subject-Fields-Index-and-Display
The text was updated successfully, but these errors were encountered: