sul-dlss-deprecated / dor_indexing_app

An indexing API for Stanford's Digital Object Repository
https://sul-dlss-deprecated.github.io/dor_indexing_app/
Apache License 2.0
0 stars 2 forks source link

Argo blacklight configuration and Solr indexing - any simplifications from #998? #999

Closed ndushay closed 11 months ago

ndushay commented 1 year ago

998 is about winnowing the below down to 2 values:

https://github.com/sul-dlss/argo/blob/72ee31b069007813b57ed42971a7349595188de9/app/controllers/catalog_controller.rb#L156-L177

image

The work of this ticket is to determine if

For example, do we make use of "SolrDocument::FIELD_COPYRIGHT" or "SolrDocument::FIELD_USE_STATEMENT" elsewhere? Do we need these declarations to do something with Blacklight, or were they created only for the Empty Fields facet? I tried to chase the code paths but I got confused in the midst of Presenters and other forms of indirection.

The less noise/useless code/configurations we have in Argo and in our Solr indexing, the better.

andrewjbtw commented 1 year ago

It's possible that those fields are used in the Profile view: https://argo.stanford.edu/profile?f%5Bexploded_tag_ssim%5D%5B%5D=Project+%3A+H2

ndushay commented 11 months ago

@jmartin-sul - please focus only on the config stuff in catalog_controller, as I have been perusing the indexing code at this time.

jmartin-sul commented 11 months ago

@jmartin-sul - please focus only on the config stuff in catalog_controller, as I have been perusing the indexing code at this time.

will do!

ndushay commented 11 months ago

I didn't find anything easy to remove, so closing this.