crkn-rcdr / cap

Public front-end for the Canadiana Access Platform
https://www.canadiana.ca
0 stars 0 forks source link

"annual" vs "annuals"? #142

Closed RussellMcOrmond closed 1 year ago

RussellMcOrmond commented 1 year ago

I thought the "id" and "subcollections" in portal configurations referenced collections in our IIIF data.

I note https://github.com/crkn-rcdr/cap/blob/main/CAP/conf/portal/online.json references something called "annual", but when I type that in I only find https://access.canadiana.ca/object/edit/69429/s04x54f1nj9z with slug "annuals" (note the 's' at the end)

Not sure if the slug is wrong, the config is wrong, or my understanding of the link between the two is wrong.

I suspect @brittnylapierre will need to do some work, but I believe @BeS4 will need to help make sense of which is correct.

RussellMcOrmond commented 1 year ago

Did some looking into this.

Short-form is that the slugs in the IIIF editor must match the 'portal' or 'subcollections' ID listed in https://github.com/crkn-rcdr/cap/tree/main/CAP/conf/portal

"annual" and "annuals" doesn't match, and we need to fix one to match the other.


12,745 total results.

3,153 total results.

I picked the top result for "annual" https://www.canadiana.ca/view/oocihm.8_01841_2/1

Then loaded https://access.canadiana.ca/object/edit/69429/m0tq5r78b64x

Last data transfer: Started: 2022-02-26T15:52:44Z

So, I forced data transfer.

It is now in

"collection": [ "annuals", "online", "serials" ],

If you look at https://www.canadiana.ca/view/oocihm.8_01841_2/1 now you will no longer see "annuals" listed for collection.

RussellMcOrmond commented 1 year ago

CAP config change: https://github.com/crkn-rcdr/cap/commit/8c5605caa1261e725b4c8058fd45c5544bf9d8b6

I need to cause cache to generated such that everything currently in "annual" will show up in "annuals".

Some of this cache generation will be happening over the Winter break.

RussellMcOrmond commented 1 year ago

The data is being migrated (Specifically, Solr re-indexing is ongoing).

A version of CAP with this change needs to go through the regular test/demo process, so passing ticket to Brittny.

RussellMcOrmond commented 1 year ago

Apparently, that already happened in December (checked the file in the image currently in production).

Closing.