hyku icon indicating copy to clipboard operation
hyku copied to clipboard

:bug: Search tenant not working with HYRAX_FLEXIBLE=true

Open kirkkwang opened this issue 6 months ago • 3 comments

Steps to reproduce:

  1. create a search tenant
  2. visit the newly created tenant
Image

kirkkwang avatar Sep 22 '25 22:09 kirkkwang

I deployed and tested your fix on hykudemo.org

Creating a new search tenant works on a flexible enabled environment ✅

But I think we need to follow up and hide metadata profiles from the search tenant as well. There shouldn't be a need for them to upload profiles since they can't create works.

Metadata profiles should be hidden from the sidebar and the route, for search only tenants.

Would you be able to complete this task? cc @trmccormick

Image Image

ShanaLMoore avatar Sep 25 '25 16:09 ShanaLMoore

Ok i have a solution in place but testing with rspec locally i'm seeing errors I'm not sure if it's my setup but the solution is working. it's at https://github.com/samvera/hyku/tree/fix-metadata-profiles-search-only-tenant/app if you want to take a look at my solution

Image Image

trmccormick avatar Sep 30 '25 16:09 trmccormick

Updated from Hyku IG call:

Per Rob, we don't need to gracefully handle the error if a user manually puts in the url. so removing the link in the sidebar will suffice @trmccormick

There may be existing code somewhere that hides things conditionally as a search only tenant. Maybe we can leverage that.

ShanaLMoore avatar Oct 23 '25 16:10 ShanaLMoore