Bump models to include new video captions field

[RikRootsGuardian]

What does this change?

Composer users need an easier way to update captions for video atoms. The current process of updating them in MAM is laborious and prevents us from showing different captions for the same video on different articles, which is sometimes helpful.

The work in this PR is part of a chain of PRs to add a new caption field to the atom editor in Composer. This field should:

• Default to the Media atom's title value, set in the Media Atom Maker tool
• Be editable within Composer
• Allow users to clear the field, which would remove the caption entirely from videos used in an article (including standfirst videos)

See the Rethink media atom captions in Composer issue (in the Arts&Pubs project) for further details

• flexible-model [includes release] - testing in CODE complete
• flexible-content - testing in CODE complete
• content-api-models [includes release] - testing in CODE complete
• content-api (Porter) - testing in CODE complete
• content-api (ES mappings) - testing in CODE complete
• content-api (Concierge)
• content-api-scala-client [includes release]
• content-api-firehose-client [includes release]
• facia-scala-client [includes release]
• frontend ← ← ← We are here
• dotcom-rendering
• apple-news

Testing

We have tested this by running frontend locally and also running DCR locally and then visiting:

http://localhost:9000/science/grrlscientist/2012/aug/07/3.json?dcr

And observing caption is being populated correctly in the relevant MediaAtomBlockElement element.

Before	After

Deployment

Some PRs include model pre-releases. Before deploying to PROD, these pre-releases will need to become full releases, and code in affected PRs updated to reflect these changes.

TODO: come up with an appropriate sequencing for merging and closing PRs. The PRs do not need to all be deployed at the same time, but some - such as those requiring associated releases - need to be performed before others.

---

Checklist

• Tested locally, and on CODE if necessary
• Will not break dotcom-rendering
• Any new test data/database files generated by tests are committed with this PR (the tests will fail in CI if you've forgotten to do this)
• Meets our accessibility standards
  • Tested with screen reader
  • Navigable with keyboard
  • Colour contrast passed

[RikRootsGuardian]

Concerning the build error:

... I think I need to perform a pre-release of the content-api-scala-client library and update the capiVersion value in the project/Dependencies.scala file ...?

[github-actions]

Deploy build 6976 of dotcom:frontend-all to CODE

All deployment options

• Deploy build 6976 of dotcom:frontend-all to CODE
• Deploy parts of build 6976 to CODE by previewing it first
• What's on CODE right now?

---

From guardian/actions-riff-raff.

[RikRootsGuardian]

Current build failure is due to a Scala formatting issue:

[simonbyford]

This PR just needs to transform a contentatom from CAPI into a MediaAtomBlockElement which DCR can render. Take a look at this PR which does something very similar:

#28616

In our case we'll need to pass in something like:

element.contentAtomTypeData.flatMap(_.caption)

[simonbyford]

We shouldn't need to touch this part of the system which deals with email templates

[RikRootsGuardian]

Removed change

[simonbyford]

I've included some testing instructions in the PR description which should help check whether this change has worked correctly!

[simonbyford]

I've been trying to understand the purpose of this file and I believe it is no longer used.

According to this comment, transformation logic used to belong in BlockElement.scala but now lives in PageElement.scala.

Therefore I suggest we omit this change.

[RikRootsGuardian]

Actioned

[RikRootsGuardian]

I agree that this is ready to merge, following the successful testing

[gu-prout]

Seen on ADMIN-PROD (created by @RikRootsGuardian and merged by @simonbyford 21 minutes and 44 seconds ago)

• Check your changes on www.theguardian.com ✔️
• Keep an eye on the deploy dashboard 📉

[gu-prout]

Seen on FRONTS-PROD (created by @RikRootsGuardian and merged by @simonbyford 21 minutes and 59 seconds ago)

• Check your changes on www.theguardian.com ✔️
• Keep an eye on the deploy dashboard 📉