-
Notifications
You must be signed in to change notification settings - Fork 10.6k
[IMP] Payroll: Updating main doc #14187
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
d1bcd1a to
440bdb7
Compare
440bdb7 to
41e3cfc
Compare
41e3cfc to
8b52a38
Compare
|
Hi @jero-odoo - this is ready for a review! |
jero-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.
Approved with comments @larm-odoo
8b52a38 to
72767a7
Compare
|
Hi @Felicious - this is ready for a final review =) |
Felicious
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.
LGTM @larm-odoo !
I'd merge directly but I noticed that you said you're planning to merge this after some other docs are merged? So I'll let you merge
@robodoo delegate+
72767a7 to
22772ca
Compare
|
@robodoo r+ |
closes #14187 Signed-off-by: Lara Martini (larm) <[email protected]>

Updating to new standards.
I'm placing this on hold. I'm updating the other Payroll docs to move relevant info from her to their corresponding docs. I'll work on this when all the other Payrol docs are done.