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 Default Locale Option to enable x-default #166

Merged
merged 5 commits into from
Jan 26, 2024

Conversation

DanielDarrenJones
Copy link
Contributor

@DanielDarrenJones DanielDarrenJones commented Jan 18, 2024

Added a new general config option to select a site as the default locale, this will then add the x-default hreflang tags to the html.

Screenshot 2024-01-18 at 14 44 46

See: https://developers.google.com/search/blog/2013/04/x-default-hreflang-for-international-pages

Implements: #82

Please let me know if you would like this done in a different way or need any amendments, this is a first draft just to get the ball rolling.

@joshuanoyes
Copy link
Contributor

@DanielDarrenJones Thanks for the contribution! All seems to work well - just a couple of small changes requested 🙂

@DanielDarrenJones
Copy link
Contributor Author

@joshuanoyes Sorry if I am being silly, but I don't seem to be able to see the requested changes?

src/Tags/AardvarkSeoTags.php Outdated Show resolved Hide resolved
src/Tags/AardvarkSeoTags.php Outdated Show resolved Hide resolved
src/Modifiers/ParseLocaleModifier.php Outdated Show resolved Hide resolved
@joshuanoyes
Copy link
Contributor

@DanielDarrenJones Sorry, forgot to press "Submit review"!

@DanielDarrenJones
Copy link
Contributor Author

@joshuanoyes both of these should be resolved now, as strict type checks, thanks!

@joshuanoyes joshuanoyes merged commit 33397ec into WithCandour:master Jan 26, 2024
@joshuanoyes
Copy link
Contributor

@DanielDarrenJones Thanks - we will include this in a release on Monday 🙂

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