-
Notifications
You must be signed in to change notification settings - Fork 10.4k
[IMP] website: hide a dynamic menu to headers and footers #14691
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
Hello @auva-odoo, I adapted a little regarding what you suggested in the task. We can discuss this together! See ya 👋 |
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.
@masp-odoo thank you for your work! Let me know if you have any questions or things you'd like to discuss!
content/applications/websites/website/structure/header_footer.rst
Outdated
Show resolved
Hide resolved
content/applications/websites/website/structure/header_footer.rst
Outdated
Show resolved
Hide resolved
content/applications/websites/website/structure/header_footer.rst
Outdated
Show resolved
Hide resolved
59e65f0
to
aaf7e59
Compare
Thank you for your review and suggestions @auva-odoo, I hope it's good to go now 💪 |
aaf7e59
to
daa6c2e
Compare
daa6c2e
to
d8bbe98
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.
Thank you for the changes @masp-odoo! I force-pushed uniformize verbs in heading, I hope you don't mind!
@robodoo r+
@masp-odoo @auva-odoo unable to stage: merge conflict |
d8bbe98
to
0f64682
Compare
@robodoo r+ |
task-5003509 closes #14691 Signed-off-by: Audrey Vandromme (auva) <auva@odoo.com>
task-5003509