-
Notifications
You must be signed in to change notification settings - Fork 9.9k
[IMP] Time Off: Updating Allocations #14181
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 @slinkous - this is ready for a peer review. Thanks! |
Hi @jero-odoo - I am reassigning this to you since Zaz is out. 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.
Looks good to me @larm-odoo just two quick notes
9dcaf34
to
c9b70ba
Compare
Hi @Felicious - this is ready for a final review/tech 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, @larm-odoo ! I just noticed some minor and random things in my review (: After picking and choosing which suggestions are helpful to you, this is ready for merge!
@robodoo delegate+
c9b70ba
to
3f32332
Compare
3f32332
to
5914a24
Compare
@robodoo r+ |
Updating to new standards, changed images and added an example admonition.
Original task card for this PR.