[ADD] Recruitment: Team performance reporting #10984
Conversation
|
|
a48d04f to
aa5896c
Compare
|
Hi @pabr-odoo - this is ready for a peer review! |
pabr-odoo
left a comment
There was a problem hiding this comment.
Hey @larm-odoo great job on this doc! The uses cases are an excellent addition. I left a handful of suggestions, so feel free to move this on to final review once you've given it a look.
aa5896c to
76eeef0
Compare
|
Hi @ksc-odoo , this is ready for a final review! |
ksc-odoo
left a comment
There was a problem hiding this comment.
hey @larm-odoo -- just finished my Final Review. Nice work on this PR! Only have a small handful of comments that require your attention. Once you take care of the necessary adjustments, feel free to tag this for Tech Review. Thanks! 👍
76eeef0 to
a51c28c
Compare
|
Hi @samueljlieber - this is ready for a tech review =) |
samueljlieber
left a comment
There was a problem hiding this comment.
Hi @larm-odoo, nice work on this new reporting doc for Recruitment!
I am requesting a few changes, please tag me for one more quick look after the following is addressed :)
team_performance.rstis missing from the recruitment.rst TOC tree and seealso. See this build error: https://runbot.odoo.com/runbot/build/68185235- None of the images are compressed in
team_performance/
a51c28c to
c5e5918
Compare
|
OMG I didn't compress the pics after you showed me yesterday! Everything should be fixed now- I swear I saw the green checkmark though- doh! Ready for another look @samueljlieber. |
samueljlieber
left a comment
There was a problem hiding this comment.
Looks good @larm-odoo, thank you!
.....
@robodoo r+
Adding the Team Performance reporting doc as it does not yet exist.
Original task card for this PR