-
Notifications
You must be signed in to change notification settings - Fork 10.4k
[ADD] Payroll: New Salary doc #14806
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 @jero-odoo - this is ready for a peer review. 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.
All set @larm-odoo ! Nice work, this one was a beast!
16bdc20 to
c4390f6
Compare
|
Hi @Felicious - this is ready for a final review! |
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.
Great work with this PR, @larm-odoo !
Almost all of my review was optional suggestions for brevity, so feel free to take whichever suggestions best fit (:
@robodoo delegate+
c4390f6 to
dfd9c1e
Compare
|
@robodoo r+ |
closes #14806 Signed-off-by: Lara Martini (larm) <larm@odoo.com>

Moving the salary info to a new doc from the main payroll doc.
Original task card for this PR.