-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
[ADD] events: event templates doc #9076
Conversation
dca6a45
to
202c9a4
Compare
202c9a4
to
a7713b1
Compare
Alrighty, @Felicious -- this one is ready for a Peer Review whenever you get a chance. As mentioned in the initial description of this PR, there are build errors because a different PR needs to be merged/published first (and its referenced in this PR)... but other than that, it should be good to go 👍 |
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.
Awesome work on this document, @ksc-odoo!
It provides a comprehensive overview of the Booth tab, which hasn't been covered in the Events documentation before. I really appreciate how you've kept the document concise by linking to relevant resources, ensuring we avoid any duplicate information across our docs.
I just had some minor rewording changes for you to consider, so after taking a look at those, feel free to move this onto the next stage of review!
092a8ac
to
6022bb6
Compare
Thanks for the super helpful feedback, @Felicious -- just implemented all your changes, and believe this one is now ready for Final Review, @StraubCreative 👍 |
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 @ksc-odoo
I thought this was well-written and pleasant to read, nice job 👍
There's only a few comments for you below but they're all high-level and optional, for you to consider. In any case, approving for the next round, so feel free to tag SALI when you're ready, thanks!
a9de63b
to
11a0605
Compare
Thanks for the helpful feedback, @StraubCreative -- I implemented all your suggestions, and believe this is now ready for Tech Review, @samueljlieber |
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.
content/applications/marketing/events/event_templates/create-booths-popup.png
Outdated
Show resolved
Hide resolved
content/applications/marketing/events/event_templates/event-templates-page.png
Outdated
Show resolved
Hide resolved
11a0605
to
0c20f3f
Compare
@robodoo r+ |
PROJECT TASK: https://www.odoo.com/web#id=3901937&cids=3&menu_id=4720&action=333&active_id=3835&model=project.task&view_type=form
This BRAND NEW PR will focus on everything the user should know about Event Templates
This PR is also adhering to some recent feedback I've received about getting too granular with explanations, so I am attempting to be more concise with the instructions presented throughout.
Also, this PR should be merged/published AFTER #8929 , because there is a lot of referenced links used in this PR, which will likely result in continued build errors until that create_events doc is published.