-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[ADD] Payroll: Working Schedules #14826
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 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.
All set @larm-odoo ! Great work!
5d1b5ef to
2b54d3f
Compare
|
Hi @Felicious - this is ready for a final review =) |
|
Hello @jeroenstijven - can you please let me know specifically what needs to be edited? The image is difficult for me to read clearly, so I am not sure what needs to be updated. Could you provide me the specific edits? Thank you! |
e6a64f3 to
758354c
Compare
|
Hi @Felicious - it appears I was tagged in error. I checked the mention above, but it all relates to accounting localization documents, so I think this was in error. I cannot see anything I need to update on this, and I have not received a reply, so this is good for you to review! I had some merge errors after merging 3 docs today - I think it is ready, but there may be another conflict popping up overnight. If so, I'll fix it! But for now, the review of the content is ready =) |
758354c to
f7c1793
Compare
|
And all the checks finally passed, @Felicious ! |
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.
@robodoo r+
closes #14826 Signed-off-by: Felicia Kuan (feku) <feku@odoo.com>

Moving the section of creating working schedules in the main payroll doc to a new doc. Also, updating links in a variety of docs referencing this. I discovered payroll sometimes refers to working schedules as 'working time' so that was also updated.
Original task card for this PR.