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

Adds vocabulary column to subject table #3386

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

christinach
Copy link
Member

@christinach christinach commented Jan 27, 2023

closes #1982
closes #1376

update multiple genre vocab
Update the browse_facet task to check for subject facet update browselists load_facet to look for subject facet Include fixtures
Updates subjects.sorted

@christinach
Copy link
Member Author

@mzelesky and I reviewed the search results of this PR. It seems to be a little bit more complicated and we should discuss this in the Orangelight meeting on 2/9/2023.

@coveralls
Copy link

coveralls commented Feb 16, 2023

Coverage Status

coverage: 96.508% (-0.03%) from 96.536%
when pulling b8e66ff on authority-vocab-updates-v2
into b0ebcb1 on main.

@pulbot pulbot temporarily deployed to qa February 16, 2023 16:35 Inactive
@christinach
Copy link
Member Author

christinach commented Feb 16, 2023

@mzelesky and I reviewed the search results of this PR. It seems to be a little bit more complicated and we should discuss this in the Orangelight meeting on 2/9/2023.

In the OL/Bibdata meeting we discussed that we need to support browselists with the additional subject and vocabulary searches that we've added in the recent years.

  • We would need to create additional tickets in bibdata to support these changes.

update multiple genre vocab
Update the browse_facet task to check for subject facet
update browselists load_facet to look for subject facet
Include fixtures
Updates subjects.sorted

Co-authored-by: Nikitas Tampakis <nikitas.tampakis@gmail.com>
@christinach
Copy link
Member Author

@christinach when I update this PR put back the .sorted

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