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

feat(seo): add support for alternate links #26

Merged
1 commit merged into from
Oct 21, 2020

Conversation

0xpatrickdev
Copy link
Contributor

@0xpatrickdev 0xpatrickdev commented Oct 20, 2020

- add support for <link rel=alternate /> tags
- can be used to point to pages in alternate languages and locales
- https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes/rel#attr-alternate
@0xpatrickdev 0xpatrickdev requested a review from a team as a code owner October 20, 2020 01:30
export const alternateLinkShape = shape({
hreflang: string,
href: string,
});
Copy link
Contributor Author

@0xpatrickdev 0xpatrickdev Oct 20, 2020

Choose a reason for hiding this comment

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

More attributes are supported with <link rel="alternate" />. Let me know if I should update the PR to include more. My thinking is that hreflang and href are the two most relevant for SEO.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Awesome

@ghost ghost merged commit af7b5a3 into americanexpress:master Oct 21, 2020
oneamexbot added a commit that referenced this pull request Oct 21, 2020
# [2.3.0](v2.2.1...v2.3.0) (2020-10-21)

### Features

* **seo:** add support for alternate links ([#26](#26)) ([af7b5a3](af7b5a3))
@oneamexbot
Copy link
Contributor

🎉 This PR is included in version 2.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

This pull request was closed.
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.

4 participants