-
Notifications
You must be signed in to change notification settings - Fork 7.2k
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
[IMP] events: update track and manage talks #9223
Conversation
77cbd32
to
babbc87
Compare
babbc87
to
fbbdc47
Compare
Hey @brse-odoo -- this one is ready for Peer Review whenever you get a chance...Thanks! 👍 |
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.
@ksc-odoo I've finished my review of this PR, great job! Very informative and easy to follow. I'm approving with some suggestions for you to accept/reject as you see fit. Let me know if you have any questions, thanks!
fbbdc47
to
78beba2
Compare
Thanks for the super-helpful feedback, @brse-odoo -- made all your suggested edits 👍 This one is now ready for Final Review @StraubCreative -- whenever you get a chance. Thanks! 🙏 |
78beba2
to
8bd319f
Compare
8bd319f
to
d772c91
Compare
Rebase to latest on d772c91 |
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
Looks good, I like how you focused the content to be more specific to talks instead of the other topics that were included in the previous iteration (Attendees, LGR, etc.)
I just had a few comments below for you to look at. Once they're addressed to your liking feel free to move on to tech review.
Cheers!
d772c91
to
e414922
Compare
Thanks again for the awesome feedback/suggestions, @StraubCreative -- really appreciate it! I made all those necessary adjustments, and believe this one 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.
e414922
to
7e23c64
Compare
Thanks for the feedback, @samueljlieber -- made that quick adjustment, and believe this one is ready to go. Thanks again! 👍 @robodoo r+ |
PROJECT TASK: https://www.odoo.com/web#id=3915122&cids=3&menu_id=4720&action=333&active_id=3835&model=project.task&view_type=form
This PR discuss what the Talks, Talk Proposals, and Agenda event website pages entail -- and how to utilize and navigate them.
This is meant as a supplemental PR to the 'Create event tracks' doc...which is focused more on the backend elements and how to create tracks for an event.
This PR should be published/merged after the 'event_tracks' PR
That's why there is likely going to be continued Build Errors -- because that 'event_tracks' doc is referenced in the seealso at the end.
I worked off the original RST file name so that I didn't ruin any potential reference links that could be lingering elsewhere...