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

Canonical URL for individual news articles based on permalink #723

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

nickdenardis
Copy link
Member

Reason for change

Because articles can be renamed and potentially accessed at different URLs, the Canonical URL assumes the current URL is the most up-to-date and uses that URL as the canonical.

This causes a situation where the same article could be accessed at multiple URLs, each with its own canonical URL. Search engines complained the same content was pointing to different canonical URLs.

This change forces the most up-to-date title (slug) to be used as the canonical URL so search engines index the correct URL.

Reminders

  • Update package version number
  • Frontend accessibility check
  • Styleguide example in place
  • New functionality covered by tests
  • Screenshots of multiple screen sizes

Demo

Before After
Screenshot 2024-08-06 at 7 20 39 AM Screenshot 2024-08-06 at 7 21 07 AM

@coveralls
Copy link

coveralls commented Aug 6, 2024

Coverage Status

coverage: 100.0%. remained the same
when pulling 41b48a7 on hotfix/8.10.6
into 86e8f76 on master.

@nickdenardis nickdenardis merged commit 8140802 into master Aug 6, 2024
2 checks passed
@breakdancingcat
Copy link
Member

👍

@nickdenardis nickdenardis deleted the hotfix/8.10.6 branch August 28, 2024 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants