Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update /voting-in-the-uk schema #1537

Merged
merged 1 commit into from
Nov 4, 2019
Merged

Update /voting-in-the-uk schema #1537

merged 1 commit into from
Nov 4, 2019

Conversation

sihugh
Copy link
Contributor

@sihugh sihugh commented Nov 4, 2019

The content was updated on Friday so we need to update this to keep it in step. There is a diff of the content changes in Publisher.

The relevant page in the preview app is https://government-frontend-pr-1537.herokuapp.com/voting-in-the-uk

Only certain sections of the page are included in the schema, because there's a limit to the amount of content that can be included in the FAQ schema. (That's also the reason for using a separate yml config file for this rather than generating it dynamically from the content item).

The initial version of the page (with the relevant FAQ sections highlighted) is available to review if necessary). This provides a decent indication of which sections are in scope for the schema, but is now out of date.

There are a couple of existing tests to ensure that this works OK.

Speak to me or Sam Dub for further details if required.

The content was updated on Friday.  There is [a diff of the content changes](https://publisher.publishing.service.gov.uk/editions/5dbc325c40f0b670153e4e18/diff)
in Publisher.

Only certain sections of the page are included in the schema, because there's
a limit to the amount of content that can be included in the FAQ schema.
(That's also the reason for using a separate yml config file for this rather
than generating it dynamically from the content item).

The [initial version of the page](https://docs.google.com/document/d/14g7FK-v_Y5iqWcB9y5WqTUWU6JuY_eaSGAqxXSMb0Ek/edit#heading=h.9w2yx5uysegs)
(with the relevant FAQ sections highlighted) is available to review if
necessary. This provides a decent indication of which bits are included,
but is now out of date.

Speak to me or Sam Dub for further details if required.
@sihugh sihugh merged commit 82edccf into master Nov 4, 2019
@sihugh sihugh deleted the voting-update-2 branch November 4, 2019 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants