-
Notifications
You must be signed in to change notification settings - Fork 9.7k
[IMP] website: seo indexation update and magic sheet #13794
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
base: 18.0
Are you sure you want to change the base?
Conversation
53a6935
to
f3b1820
Compare
Hi @AutumnWalk, hope you're fine! Can you please review these changes? Thanks a lot! |
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.
Hi @masp-odoo , Thank you for improving this doc! I have one comment about the noindex.
Thank you
To effectively prevent a page from appearing in search engine results, use one of the following | ||
methods: | ||
|
||
- **noindex tag:** Implement a `noindex` meta tag within the HTML of the page. This explicitly |
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.
For this part, could you clarify 'noindex' meta tag within the HTML header?
- noindex tag: Implement a
noindex
meta tag within the HTML header of the page.
Side note to you, Odoo has the feature to toggle index and noindex via page properties. This only show for pages listed in Website -> Site -> Pages. For (dynamic) pages coming from a module extending website, this option is not available yet. So customer has to customize at this time if they want to have noindex for "dynamic" pages. The RD team is working on allowing this noindex toggle for all pages. https://www.odoo.com/odoo/project/974/tasks/3861418?debug=1
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.
Hi @AutumnWalk, I adapted the text, is it clear enough?
Thanks a lot for your quick review!
f3b1820
to
99612f8
Compare
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.
Sorry, I totally missed your message. Please feel free to ping me if I haven't answered you in 24 hr time.
99612f8
to
87fc92d
Compare
taskid-4478979
87fc92d
to
46ead0c
Compare
Dear @AutumnWalk, thanks again for replying so quickly, and no worries! Can you confirm that it's clear and understandable now? Thanks a lot! |
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.
This good look!
Thank you for writing this piece
taskid-4478979