Add new metadata field for imageType, and UI controls

andrew-nowak commented 2 months ago

What does this change?

Adds a new metadata field named "image type" (or imageType in code responses), to designate an image as either "Photograph", "Illustration" or "Composite".

This has been requested by the Guardian's picture desk, so we can designate images into one of these categories and have that value pull through automatically into the content publishing tools.

Requires a mapping update (no migration necessary) -- see instructions at https://github.com/guardian/grid/tree/main/scripts#updatemapping

Values are set in config, by adding a key eg. imageTypes = ["Photograph", "Illustration", "Composite"]

Currently this value is only required by Kahuna, but I would recommend setting in common config, so that the values can be read by other services, in the case of future extensions of this feature.

How should a reviewer test this change?

Deploy to TEST, and try setting the value

How can success be measured?

we will finally be able to simplify Usage Rights (as they currently conflate type and rights)
we will be able to listen to some external signals (mostly DigitalSourceType, but maybe some weaker ones like signals in Description from agencies)
we will be able to ask our partners to supply DigitalSourceType
and, most importantly, we will be able to be more transparent with our audience about the nature of the image

Who should look at this?

Tested? Documented?

[x] locally by committer
[ ] locally by Guardian reviewer
[x] on the Guardian's TEST environment
[ ] relevant documentation added or amended (if needed)

github-actions[bot] commented 2 months ago

Deploy build 12817 to TEST

All deployment options

- [Deploy build 12817 to TEST](https://riffraff.gutools.co.uk/deployment/deployAgain?project=media-service%3A%3Agrid%3A%3Aall&build=12817&stage=TEST&updateStrategy=MostlyHarmless&action=deploy) - [Deploy parts of build 12817 to TEST by previewing it first](https://riffraff.gutools.co.uk/preview/yaml?project=media-service%3A%3Agrid%3A%3Aall&build=12817&stage=TEST&updateStrategy=MostlyHarmless)

From guardian/actions-riff-raff.

andrew-nowak commented 2 months ago

TODO:

[x] multiple select
[x] set to illustration when usage right is set to illustrator
[x] set to composite when usage right is set to composite