-
Notifications
You must be signed in to change notification settings - Fork 10.6k
[IMP] website: "pages" page #4871
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
Conversation
b98f42f to
5e12a68
Compare
5063c77 to
ce49002
Compare
LoredanaLrpz
left a comment
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.
Hey, @auva-odoo , great work overall =D
Here's my review
Please note that I usually don't check the count of characters when I make suggestions x)
Also, don't forget to change the commit msg since you co-authored the PR
--> commit --amend and write
Co-authored-by: Toaa <[email protected]>
Co-authored-by: Auva <[email protected]>
ce49002 to
d906b40
Compare
d906b40 to
03e6270
Compare
LoredanaLrpz
left a comment
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.
cc
xpl-odoo
left a comment
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.
Nice job Audrey/Tom! :) Don't hesitate to ask questions regarding my suggestions or challenge them ;)
Adding a documentation on "pages" within the already existing "page" toctree task-3383323 Co-authored-by: Toaa <[email protected]> Co-authored-by: Auva <[email protected]>
03e6270 to
ccb2564
Compare
xpl-odoo
left a comment
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.
@robodoo delegate+
|
@robodoo r+ |
|
I'm sorry, @auva-odoo. I'm afraid I can't do that. |
|
@robodoo r+ |
Adding a documentation on "pages" within the already existing "page" toctree
task-3383323
@auva-odoo Hi Audrey, could you have a - thorough - look at this PR? I wrote it based on your notes, but I could not understand everything. You'd know better what is supposed to be in here :)