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

Clicking "view all collections" on nurax-pg homepage returns no results #5567

Closed
rjkati opened this issue Mar 21, 2022 · 5 comments · Fixed by #5614
Closed

Clicking "view all collections" on nurax-pg homepage returns no results #5567

rjkati opened this issue Mar 21, 2022 · 5 comments · Fixed by #5614

Comments

@rjkati
Copy link

rjkati commented Mar 21, 2022

Descriptive summary

On the nurax-pg homepage, clicking the "View All Collections" button returns no results, even though the "Explore Collections" list displays multiple collections.

Expected behavior

When collections appear in the "Explore Collections" list, they should appear in a "View All Collections" search.

Actual behavior

No collections appear when clicking "View All Collections"

Steps to reproduce the behavior

  1. Navigate to https://nurax-pg.curationexperts.com/?locale=en
  2. Click the "View All Collections" button in the "Explore Collections" tab.
  3. Observe that no results have been returned.
@elrayle
Copy link
Contributor

elrayle commented Apr 4, 2022

In some places, the search for collections uses generic_type and in others uses human_readable_type. If the filter uses generic_type, then the list of collections are displayed.

Testing in SOLR:

  • human_readable_type_sim:Collection returns collections in nurax-dev, but not nurax-pg
  • human_readable_type_tesim:Collection returns collections in both
  • generic_type_sim returns collections in both

To maintain backward compatibility, the solr document for Hyrax::PcdmCollections should include human_readable_type_sim:Collection.

@no-reply
Copy link
Contributor

why'd we choose human_readable_type instead of generic_type?

the latter seems more fit for purpose, and i'm not following how the former is more backwards compatible.

@jlhardes
Copy link
Contributor

This problem is still occurring on nurax-pg.

@rjkati
Copy link
Author

rjkati commented May 23, 2023

This is still occurring on pg.nurax running Hyrax 4.0.0.rc2

alishaevn added a commit that referenced this issue Jun 5, 2023
@gamontoya
Copy link

@rjkati This can be closed. It's working fine now on nurax-pg Hyrax Version: 4.0.0

@rjkati rjkati closed this as completed Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants