Skip to content

[IMP] Website: update pages page #13605

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

Open
wants to merge 1 commit into
base: 18.0
Choose a base branch
from
Open

Conversation

guga-odoo
Copy link

task-4459299

@robodoo
Copy link
Collaborator

robodoo commented Jun 3, 2025

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team June 3, 2025 14:40
@guga-odoo guga-odoo requested review from a team and removed request for a team June 4, 2025 06:18
@auva-odoo auva-odoo requested review from auva-odoo and removed request for a team June 4, 2025 12:12
Copy link
Contributor

@auva-odoo auva-odoo left a comment

Choose a reason for hiding this comment

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

Hello @guga-odoo, thank you so much for your work on this! I added a few comments and suggestions (a few of them are actually about the same things, but I felt it was clearer if I repeated them 😉 )

Also, in general, I use the following rules to decide whether the first letter after a colon (in a list item describing a field for example) should be capitalized and should have a period at the end:

  1. Capitalize the first letter and add a period at the end if it's a full sentence.
  2. Don't capitalize and don't use a period if it's not (in this case you could use a semi-colon at the end, not mandatory though)
  3. If some items in the list do have a full sentence after the colon and some don't, uniformize (i.e. either use capitals/periods/semi-colons or don't use them, but don't use a mix of both in the same list). Consistency is key 🙂

I didn't add individual comments for this in this PR but could you please go through the doc and check all list (items) this might apply to? (there are a few exceptions in this doc, ofc)

Thank you!!

@@ -15,6 +15,8 @@ other hand, are generated dynamically. All pages generated automatically by Odoo
you install an app or module (e.g., `/shop` or `/blog`) or publish a new product or blog post, are
dynamic pages and are therefore managed differently.

.. _website/page_creation:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
.. _website/page_creation:
.. _website/pages/page_creation:

(nitpicking, to fit our standard anchor names: app name/file name/ anchor name to try and make sure anchor names stay unique throughout the whole doc long-term 😉 )

Also, I can't comment on this because you didn't change it but:

  • I would change the anchor name on line 10 in the same way.
  • I would add a ref (to the homepage section) on the word homepage on line 12.
  • Same for blog post on line 15 (I would use a doc link there that refers to the blog doc page).

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.

3 participants