-
Notifications
You must be signed in to change notification settings - Fork 3
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
[PR-22094] Fix docs.talkable.com SEO critical issues #328
base: master
Are you sure you want to change the base?
Conversation
@@ -183,7 +183,7 @@ | |||
|
|||
# The URL which points to the root of the HTML documentation. | |||
# It is used to indicate the location of document like canonical_url. | |||
html_baseurl = 'https://docs.talkable.com' | |||
# html_baseurl = 'https://docs.talkable.com' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this commented out?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zhuravel html_baseurl added a link attribute to the main page with canonnical link 'https://docs.talkable.com/index.html', what we want actually is 'https://docs.talkable.com', to achieve this, we moved this logic to linktags.html
Co-authored-by: Bohdan Zhuravel <[email protected]>
Demo
https://deploy-preview-328--talkable-docs.netlify.app/
Related Stories