wellcomecollection / content-api

đź“– The API + ETL pipeline for searching the Wellcome Collection Prismic Repository.
MIT License
0 stars 0 forks source link

Audit addressable content types for descriptions #152

Open rcantin-w opened 1 week ago

rcantin-w commented 1 week ago

Audit out which content type uses what in reality, and which should be:

  1. Displayed on the UI
  2. Query-able. The thing to consider here would be that they shouldn't necessarily all be appended as it might skew the relevance.
davidpmccormick commented 4 days ago

I think metadata description was primarily used to improve SEO. My instinct is that this would then be less appropriate as human-friendly search content?

davidpmccormick commented 4 days ago

Having looked through a sample of the content types and the fields mentioned above for each, I propose the following.

The metadata description is not consistently populated anywhere and I think it should be ignored for this piece of work altogether.

Exhibition

Event

Story

Page

Visual Story

Exhibition text

Exhibition highlight tour

Book

Project

Season

rcantin-w commented 3 days ago

Could you update the table with your findings 🙏 ? https://www.notion.so/wellcometrust/All-search-definition-acceptance-criteria-1366687658a180f28254d4429bf764ce

rcantin-w commented 3 days ago

How is this affected by the standfirst changes ticket and do we think it's something we should consider doing before doing this work?

davidpmccormick commented 3 days ago

It would be affected if/when we do it, yes. My instinct is to carry on with this work before making changes to the standfirst – we already have queries using the current standfirst-as-a-slice implementation and they will need to be updated as well. Then they can all be updated at once.

rcantin-w commented 3 days ago

Sounds good. We'll need to make sure the Content API supports both until it gets fully removed, but I think after this piece of work, we'll always have to think "How does my change affect the Content API" anyway.

LaurenFBaily commented 2 days ago

Reviewed and this all makes sense to me, minus the event question which is captured on a separate ticket anyway.