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 'Census 2021' suffix to unauthenticated page titles #258

Merged
merged 8 commits into from
Aug 28, 2020

Conversation

petechd
Copy link
Contributor

@petechd petechd commented Aug 27, 2020

What is the context of this PR?

This hardcodes all unauthenticated page titles to include '- Census 2021' suffix. The change is described on this Trello card.

How to review

Ensure all unauthenticated pages have this suffix now.

Checklist

  • New static content marked up for translation
  • Newly defined schema content included in eq-translations repo

@petechd
Copy link
Contributor Author

petechd commented Aug 27, 2020

Copy link
Contributor

@MebinAbraham MebinAbraham left a comment

Choose a reason for hiding this comment

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

The privacy and accessibility pages needs updating. At the moment they use the survey title but it probably should just be:

  • Privacy and data protection - Census 2021
  • Accessibility statement - Census 2021

Worth clarifying with Laura.

@petechd
Copy link
Contributor Author

petechd commented Aug 27, 2020

The privacy and accessibility pages needs updating. At the moment they use the survey title but it probably should just be:

  • Privacy and data protection - Census 2021
  • Accessibility statement - Census 2021

Worth clarifying with Laura.

I've just checked with Laura and she confirmed it.

@petechd petechd merged commit 35829a5 into master Aug 28, 2020
@petechd petechd deleted the add-census-suffix-to-pages branch August 28, 2020 12:21
@pricem14pc pricem14pc added this to the v3.48.0 milestone Sep 11, 2020
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.

5 participants