-
Notifications
You must be signed in to change notification settings - Fork 10.9k
[ADD] Employees: Employee retention report #11045
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
|
Hi @kekiss - this is ready for a first-round review! |
kekiss
left a comment
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.
Fantastic job Lara! I had one small note, but aside from that, everything looked great. Feel free to pass this on to final review once you check out my suggestion!
b3d7593 to
120491a
Compare
|
Hi @ksc-odoo - this is ready for a final review! |
ksc-odoo
left a comment
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 @larm-odoo -- just finished my Final Review. Only a small collection of comments that require your attention. Once you implement the necessary adjustments, feel free to move this to Tech Review. Thanks
120491a to
f836655
Compare
|
Hi @samueljlieber - this is ready for a tech review! |
samueljlieber
left a comment
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 @larm-odoo, nice work on this new reporting doc for Employees. I have a small number of changes before merge. Please see below!
Thank you for your work!
.....
@robodoo delegate=larm-odoo
content/applications/hr/employees/retention_report/by-department.png
Outdated
Show resolved
Hide resolved
f836655 to
059ebdc
Compare
|
@robodoo r+ |

No documentation relating to employee retention reporting exists.
Original project task card for this PR.