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

Updates for How To Vote guide #2034

Merged
merged 6 commits into from
Mar 5, 2021
Merged

Updates for How To Vote guide #2034

merged 6 commits into from
Mar 5, 2021

Conversation

sihugh
Copy link
Contributor

@sihugh sihugh commented Mar 5, 2021

We're going to run https://www.gov.uk/how-to-vote as a single page guide up to the local elections. This allows us to use the FAQPage schema to extend the reach of the content of this page to other platforms.

We wrote about doing this for the general election.

This PR resurrects the old way of doing this, by:

  • reverting two old commits
  • updating the slugs (because it used to be called /how-to-vote
  • updating the content from the document agreed with stakeholders
  • fixing a failing test due to changes in stubbing methods

Note, we may want to review the single page guide template, as some elements of GOV.UK frontend have moved on.

https://government-f-voting-int-jrhxkr.herokuapp.com/how-to-vote

⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️

sihugh added 3 commits March 5, 2021 15:04
The old voting-in-the-uk guide has moved.
@bevanloon bevanloon temporarily deployed to government-f-voting-int-jrhxkr March 5, 2021 15:50 Inactive
@bevanloon bevanloon temporarily deployed to government-f-voting-int-jrhxkr March 5, 2021 16:15 Inactive
@sihugh sihugh force-pushed the voting-intent-schema branch from 29d27ae to fc8bce7 Compare March 5, 2021 16:18
@bevanloon bevanloon temporarily deployed to government-f-voting-int-jrhxkr March 5, 2021 16:19 Inactive
Copy link
Contributor

@alex-ju alex-ju left a comment

Choose a reason for hiding this comment

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

Looks good overall. Only one content-related comment to be checked. I don't mind longer pages as long as users can navigate to them using the contents list and share direct links to those sections. I get that this is a trial and we can make the 'bespoke' code more generic if we decide to move more guides to this model.

config/routes.rb Outdated Show resolved Hide resolved
config/machine_readable/how-to-vote.yml Outdated Show resolved Hide resolved
app/views/content_items/guide_single.html.erb Show resolved Hide resolved
app/views/content_items/guide_single.html.erb Show resolved Hide resolved
sihugh added 3 commits March 5, 2021 16:46
This code was recently resurrected, and it refers to the old way of stubbing
using the gds-api-adapters test helpers. Fixed!
This indicates what we think the most common related services are.
@sihugh sihugh force-pushed the voting-intent-schema branch from fc8bce7 to b7758aa Compare March 5, 2021 16:46
@bevanloon bevanloon temporarily deployed to government-f-voting-int-jrhxkr March 5, 2021 16:47 Inactive
@sihugh
Copy link
Contributor Author

sihugh commented Mar 5, 2021

Looks good overall. Only once content-related comment to be checked. I don't mind longer pages as long as users can navigate to them using the contents list and share direct links to those sections. I get that this is a trial and we can make the 'bespoke' code more generic if we decide to move more guides to this model.

Totally agree 👍

@sihugh sihugh merged commit 51233ea into master Mar 5, 2021
@sihugh sihugh deleted the voting-intent-schema branch March 5, 2021 16:58
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