exclude original metadata from free text search #4056
Conversation
|
This is to resolve an issue reported by BBC News Archivists, after correcting the spelling of names in Agency wire pictures, It was not an issue for the GRID users https://journalism-tools-dev.slack.com/archives/CHYRSQ673/p1678457702896019 |
There was a problem hiding this comment.
I've tested this in our TEST environment with @paperboyo and it does work, but contrary to what we stated at grid hour today will require a 'migration' before this change applies to ANY images (i.e. including new images following this PR being merged).
We're still deciding whether we'll run a migration imminently to benefit from this ourselves, will let you know. Regardless, the offer still stands to assist BBC with their first migration (which is more pressing now we know this PR will have no effect at all until after migration).
prout-bot
added
Pending-on-cropper
Pending-on-leases
Pending-on-media-api
Pending-on-image-loader
Pending-on-metadata-editor
Pending-on-collections
Pending-on-usage
Pending-on-kahuna
Pending-on-auth
Pending-on-thrall
Seen-on-leases
Seen-on-usage
Seen-on-kahuna
Seen-on-collections
and removed
Pending-on-leases
Pending-on-collections
Pending-on-usage
Pending-on-kahuna
labels
|
Seen on collections, leases, usage, kahuna (created by @abdelrahmansd and merged by @twrichards 10 minutes and 52 seconds ago) Please check your changes! |
|
Seen on metadata-editor, thrall, cropper, media-api (created by @abdelrahmansd and merged by @twrichards 10 minutes and 59 seconds ago) Please check your changes! |
|
Seen on auth (created by @abdelrahmansd and merged by @twrichards 11 minutes and 3 seconds ago) Please check your changes! |
|
Seen on image-loader (created by @abdelrahmansd and merged by @twrichards 11 minutes and 10 seconds ago) Please check your changes! |
3 similar comments
|
Seen on image-loader (created by @abdelrahmansd and merged by @twrichards 11 minutes and 10 seconds ago) Please check your changes! |
|
Seen on image-loader (created by @abdelrahmansd and merged by @twrichards 11 minutes and 10 seconds ago) Please check your changes! |
|
Seen on image-loader (created by @abdelrahmansd and merged by @twrichards 11 minutes and 10 seconds ago) Please check your changes! |
What does this change?
update elastic search Mapping excludes original metadata from free text search (englishAnalysedCatchAll)
How should a reviewer test this change?
clean and set-up the grid locally to create the new mapping
upload an image and edit any field of originalMetadata
free text search of the old text should not get any result
How can success be measured?
Who should look at this?
Tested? Documented?