🎁Add facets for "facetable" properties

[laritakr]

related to

• 408

• 392

Todo

• https://github.com/scientist-softserv/utk-hyku/pull/463

Ideally this work will be implemented via allinson flex. As a last resort, manually set them. Add all missing facets based on metadata profile.

• [ ] Identify remaining terms to be added
• [ ] Manually add remaining terms
• [ ] Make a ticket to get it working with AllinsonFlex

Related to #426

• Potentially related to potentially related to https://github.com/samvera-labs/allinson_flex/issues/53

Testing Instructions

• [ ] Create a new work or edit an existing work
• [ ] Add faceted properties to the work, which include:
  • [ ] form_local
  • [ ] intermediate_provider
  • [ ] keyword
  • [ ] language
  • [ ] language_local
  • [ ] license
  • [ ] resource_type
  • [ ] rights_statement
  • [ ] spatial
  • [ ] subject
  • [ ] temporal
• [ ] when doing a catalog search for that work's title, observe that those fields show up on the left side as facets

Notes

It looks like Allinson flex has a switch statement to determine the solr suffix -
https://github.com/samvera-labs/allinson_flex/blob/ab9b353a3034c182218356c9320ebe0b6b952a79/app/services/allinson_flex/dynamic_schema_service.rb#L207-L227

AF may store what is facetable but it doesn't auto connect anything to the catalog controller for it to facet automatically.

Maybe check if ESSI already have an implementation for this?

[jillpe]

remaining terms are listed as facetable in the profile

[ShanaLMoore]

If AllinsonFlex isn't capable of doing this, manually add them for now and create a separate ticket for AllinsonFlex to do it automatically.

[laritakr]

facetable fields based on most recent metadata profile:

• form_local
• intermediate_provider
• keyword
• language
• language_local
• license
• resource_type
• rights_statement
• spatial
• subject
• temporal

Current facet fields in catalog_controller:

[ShanaLMoore]

Per LaRita, indexing seems to be working correctly but the dynamic catalog behavior is not being included anywhere. ESSI is including it in their catalog controller: https://github.com/search?q=repo%3AIU-LibrariesJointDevelopment%2Fessi%20DynamicCatalogBehavior&type=code

[ShanaLMoore]

@laritakr We were able to figure out #425 :tada: Turns out there's support for facets with allinson flex after all. With your tip, we figured out it wasn't being included. With Deon, we figured out it wasn't working because of a minor but significant typo. After correcting that, the facets appears as defined by the m3 metadata profile.

NEXT STEPS: @DeonFranklin will submit the typo fix to allinson flex. Once that gets merged, he will submit a PR that

1. has the module included in the catalog controller
2. updates allinson flex to pull in the fix

Thanks for driving @DeonFranklin

[DeonFranklin]

• https://github.com/scientist-softserv/utk-hyku/pull/464

[DiemBTran]

~Needs further review:~ EDIT: This passes SoftServ QA The language issues arise from the M3 profile, not the work that was done here. So this ticket passes.

tested on:

• work show page: https://qa.utk-hyku-staging.notch8.cloud/concern/books/6dc76370-9865-46ef-b49c-fcdd4c56e429?locale=en
• search catalog: https://qa.utk-hyku-staging.notch8.cloud/catalog?utf8=%E2%9C%93&locale=en&search_field=all_fields&q=2023-07-18+%F0%9F%8E%81Add+facets+for+%22facetable%22+properties+%23425

Testing Instructions

• [x] form_local
• [x] intermediate_provider
• [x] keyword
• [x] language
• [ ] language_local
  • Something is fishy about Language and Local Language. I typed in Vietnamese for Language and English for Language local. When I save the form, Language: English appears twice on the work show page and search catalog. Language: Vietnamese doesn't appear at all. Editing the work shows that Vietnamese is overridden with English.
  • screen recording: https://share.getcloudapp.com/rRuzObRD
• [x] license
• [x] resource_type
• [x] rights_statement
• [x] spatial
• [x] subject
• [x] temporal

Facetable properties appear in search facets

 

[josh-morgan117]

@laritakr for clarification, based on what I've read for the ticket, it seems like y'all have determined how to get AllinsonFlex to do this automatically based on the m3 profile instead of having to manually add them, correct?

[laritakr]

@kidon0011 Yes, we found indications that it should have been working, and found several small bugs that were causing the problem. So it works now. :)

[jillpe]

Verification on production blocked until #358 resolved