Skip to content

[IULRDC-198] bugfix: dashboard AdminSet select#67

Draft
aploshay wants to merge 2 commits into
mainfrom
iulrdc-184_dashboard_adminset_select
Draft

[IULRDC-198] bugfix: dashboard AdminSet select#67
aploshay wants to merge 2 commits into
mainfrom
iulrdc-184_dashboard_adminset_select

Conversation

@aploshay

@aploshay aploshay commented Apr 23, 2026

Copy link
Copy Markdown
Contributor

In draft status pending investigation of why this isn't happening on my localhost, and if its fixable via solr indexing

  • Admin Set appears only once in the Collection Type filtering dropdown on the Collections pages on the Dashboard. Previously it appeared twice, as “Admin Set” and as “Hyrax::AdministrativeSet”
    UPDATE: got it to happen on localhost; see follow-up comments

@rdlebeau rdlebeau changed the title [IULRDC-184] bugfix: dashboard AdminSet select [IULRDC-198] bugfix: dashboard AdminSet select Apr 27, 2026
@aploshay

Copy link
Copy Markdown
Contributor Author

@rdlebeau This appears to be a hyrax bug -- @dlpierce should we create an issue for it, in the hyrax repo? Is it still present in the current version?

My::CollectionsController configures facets with a skip method:
https://github.com/samvera/hyrax/blob/v5.0.1/app/controllers/hyrax/my/collections_controller.rb#L9-L23
that currently excludes Hyrax::ModelRegistry.collection_rdf_representations values from menu selection:
["Collection", "Hyrax::PcdmCollection"]
but it's missing a corresponding exclusion of Hyrax::ModelRegistry.admin_set_rdf_representations:
["AdminSet", "Hyrax::AdministrativeSet"]
which is why "Hyrax::AdministrativeSet" is showing up alongside "Admin Set" in the Collection Type select menu at /dashboard/collections?locale=en

Suggested fix is up on my fork:
main...aploshay:rdc:iulrdc-184_dashboard_adminset_select
Note that because the controller immediately calls .configure_facets on line 23, this has to be done as a full override instead of a patch, to work.

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