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

Improve Implementation Guide page titles and meta descriptions #4977

Merged
merged 2 commits into from
Feb 3, 2025

Conversation

andysellick
Copy link
Contributor

What

Improves page titles and adds metadata descriptions for the analytics section of the dev docs.

Why

Previously no meta descriptions, so sharing pages didn't provide enough information.

Visual changes

None.

Trello: https://trello.com/c/PmI26YFF/839-update-ga4-implementation-record-page-titles

@andysellick andysellick changed the title Analytics meta Improve Implementation Guide page titles and meta descriptions Jan 31, 2025
@andysellick andysellick requested a review from AshGDS January 31, 2025 15:00
Copy link
Contributor

@AshGDS AshGDS left a comment

Choose a reason for hiding this comment

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

Looks OK to me 👍 , but I've noticed two things:

  1. We're using the old GOVUK Crown, since you're modifying the analytics layout already would you like to include that change in this repo?
  2. The social media preview image no longer exists: https://docs.publishing.service.gov.uk/images/govuk-large.png

No worries if you think they should be done in separate PRs.

@andysellick
Copy link
Contributor Author

@AshGDS I think best done in a separate PR. Not sure who's responsible for maintenance of the docs but probably something in their remit.

Copy link
Contributor

@AshGDS AshGDS left a comment

Choose a reason for hiding this comment

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

@andysellick OK - it looks like the crown is only wrong on the analytics docs though, so it would probably come back to us. The repo is owned by #govuk-developers so I think the responsibility is for everyone to manage it. Do you want me to look into fixing these?

@andysellick
Copy link
Contributor Author

@AshGDS sorry, didn't quite get that, yes I'll take a look, thanks.

@andysellick andysellick merged commit 3d963c3 into main Feb 3, 2025
5 checks passed
@andysellick andysellick deleted the analytics-meta branch February 3, 2025 09:27
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.

2 participants