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 a description to the past prime ministers page #8523

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

leenagupte
Copy link
Contributor

@leenagupte leenagupte commented Nov 17, 2023

It was pointed out that the past prime ministers page was missing a description. This means that only the title of the page shows up in the search results.

The description has been hard-coded to match the page title, though this content should probably be moved to a locale file or be editable by publishers in the UI.

The content for the description also needs to be checked by a content designer.

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

Follow these steps if you are doing a Rails upgrade.

It was pointed out that the past prime ministers page was missing a
description. This means that only the title of the page shows up in the
search results.

The description has been hard-coded to match the page title, though
this content should probably be moved to a locale file or be editable
by publishers in the UI.

The content for the description also needs to be checked by a content
designer.
@leenagupte leenagupte force-pushed the add-past-prime-ministers-description branch from bf00705 to 63e2001 Compare November 17, 2023 18:04
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.

None yet

1 participant