Skip to content
This repository has been archived by the owner on Nov 24, 2022. It is now read-only.

Add new examples for direct attachment rendering #975

Merged
merged 1 commit into from
Mar 25, 2020

Conversation

benthorner
Copy link
Contributor

https://trello.com/c/wM9wqe9H/1587-support-featured-attachments-directly-on-govuk-frontend

Previously we relied on pre-rendered HTML for the attached 'documents'
of publications and consultations. This adds an example to support work
to directly render attachments based on metadata in the payload, and an
array of featured attachment IDs. See RFC 116 for more details.

https://github.com/alphagov/govuk-rfcs/blob/master/rfc-116-store-attachment-data-in-content-items.md

@benthorner
Copy link
Contributor Author

I did attempt to update some of the other examples, but the content changes mean this could cause random test failures wherever we're referencing the content directly.

Previously we relied on pre-rendered HTML for the attached 'documents'
of publications and consultations. This adds an example to support work
to directly render attachments based on metadata in the payload, and an
array of featured attachment IDs. See RFC 116 for more details.

Note that 'documents' is a required field for publications, so we still
need to populate it with an empty array.

https://github.com/alphagov/govuk-rfcs/blob/master/rfc-116-store-attachment-data-in-content-items.md
@benthorner benthorner force-pushed the add-featured-attachment-examples branch from 7c9c160 to 3ac9803 Compare March 25, 2020 14:36
Copy link
Contributor

@ChrisBAshton ChrisBAshton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@benthorner benthorner merged commit 48f1688 into master Mar 25, 2020
@benthorner benthorner deleted the add-featured-attachment-examples branch March 25, 2020 16:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants