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

Add hardcoded FAQPage schema for voting-in-the-uk #1527

Merged
merged 1 commit into from
Oct 25, 2019
Merged

Conversation

sihugh
Copy link
Contributor

@sihugh sihugh commented Oct 25, 2019

We think that the search intents around this page mean that users will benefit from a more curated schema. This is intended to be a one of a kind for now, and is obviously time sensitive, hence the somewhat blunt approach.

The content for the FAQPage schema is loaded from a local config file, and presented. It's hard to do this programmatically from the content in the content item because the sections we want don't necessarily map to an h2 and the following paragraphs (for example).

We'll be looking into how to structure our content/apis to make things like this easier in the near future.

I've added a couple of light touch tests to ensure that they aren't broken inadvertently.

We'll be keeping the content in the yaml file in step with the published content until we find a better approach.


https://government-frontend-pr-1527.herokuapp.com/voting-in-the-uk

@bevanloon bevanloon temporarily deployed to government-frontend-pr-1527 October 25, 2019 13:14 Inactive
@bevanloon bevanloon temporarily deployed to government-frontend-pr-1527 October 25, 2019 13:15 Inactive
@bevanloon bevanloon temporarily deployed to government-frontend-pr-1527 October 25, 2019 13:20 Inactive
@bevanloon bevanloon temporarily deployed to government-frontend-pr-1527 October 25, 2019 13:25 Inactive
@bevanloon bevanloon temporarily deployed to government-frontend-pr-1527 October 25, 2019 13:28 Inactive
@bevanloon bevanloon temporarily deployed to government-frontend-pr-1527 October 25, 2019 13:32 Inactive
@bevanloon bevanloon temporarily deployed to government-frontend-pr-1527 October 25, 2019 13:34 Inactive
@sihugh sihugh marked this pull request as ready for review October 25, 2019 13:38
We think that the voting intents around this page mean that users will benefit
from a more curated schema.  This is intended to be a one of a kind for now,
and is obviously time sensitive, hence the somewhat blunt approach.

I've added a couple of light touch tests to ensure that they aren't broken
inadvertently.

We'll be keeping the content in the yaml file in step with the published
content until we find a better approach.
Copy link
Contributor

@bilbof bilbof left a comment

Choose a reason for hiding this comment

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

Nice work!

@sihugh sihugh merged commit 878dc0c into master Oct 25, 2019
@sihugh
Copy link
Contributor Author

sihugh commented Oct 25, 2019

Thanks Bill!

@sihugh sihugh deleted the voting-faqpage branch October 25, 2019 14:59
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