-
Notifications
You must be signed in to change notification settings - Fork 7.1k
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: revenues report #9953
Conversation
a2e07ab
to
b599318
Compare
b599318
to
3df6a3b
Compare
Alrighty, @jero-odoo -- this one's ready for a 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.
Hey @ksc-odoo great work! I just had a few notes and suggestions, but otherwise this is ready for the next round. Let me know if you have any questions, thanks!
972ed0b
to
13acd92
Compare
Thanks for the super-helpful feedback, @jero-odoo -- I made those adjustments, and believe this PR is now ready for Final Review (@StraubCreative -- 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.
Hi @ksc-odoo - great doc! I learned a lot reading this, and the examples you gave are great and I think a value-add for the readers. Just a few comments, all optional (and some questions to consider).
@ksc-odoo to merge this in please address @larm-odoo feedback and pass to @samueljlieber after, thanks! |
c4a7a9c
to
027cfaa
Compare
Thank you for the helpful feedback, @larm-odoo -- I made those edits, and believe this one is now ready for Tech Review, @samueljlieber (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.
027cfaa
to
4ba4f78
Compare
@robodoo r+ |
This BRAND NEW PR specifically focuses on the Revenues portion of the Reporting element in the Odoo Events application.
This doc will succinctly cover the basics of the Revenues Reporting page, the default settings, and how to create custom reports for unique analysis.
PROJECT TASK: https://www.odoo.com/odoo/3835/tasks/4012680?cids=3