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

bug 1449210: Add hreflang for the current document #4745

Merged
merged 1 commit into from Apr 23, 2018

Conversation

Projects
None yet
2 participants
@MatonAnthony
Copy link
Contributor

MatonAnthony commented Apr 22, 2018

Google is ignoring the hreflang if the current page is not referenced
too.

This commit add the hreflang for the current page you are in top
of the already existing hreflang for the translations.

bug 1449210: Add hreflang for the current document
Google is ignoring the hreflang if the current page is not referenced
too.

This commit add the hreflang for the current page you are in top
of the already existing hreflang for the translations.
@jwhitlock
Copy link
Member

jwhitlock left a comment

Good branch name, good commit message, small and correct change. Excellent work @MatonAnthony!

@@ -69,6 +69,8 @@
<link rel="alternate" type="application/json" href="{{ url('wiki.json_slug', document.slug) | absolutify }}">
<link rel="canonical" href="{{ canonical }}" >

{# Google requires the current page to be referenced as an alternate too for SEO purposes see bug 1449210 #}

This comment has been minimized.

@jwhitlock

jwhitlock Apr 23, 2018

Member

Nice use of Jinja comments

@jwhitlock jwhitlock merged commit 78ddba4 into mozilla:master Apr 23, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.