-
Notifications
You must be signed in to change notification settings - Fork 10k
[ADD] Time Off: Add reporting #14182
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 @Felicious - since yu did the final review, and already read this in the Time Off doc, I thought I would just tag you directly for a final review. This info has been migrated from the main Time Off doc, and I added some custom anchors to the beginning paragraph, to jump to the various reports. |
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.
The content looks good to me, @larm-odoo !
The changes requested status to make a smalll edit to the custom anchors (: After you're done fixing, DM me so i can do a quick check and hopefully just merge!
dc7ed1a
to
248497e
Compare
Ready for another look @Felicious! |
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 !
@robodoo r+
Moving the reporting section from the original main Time Off doc, to its own stand-alone doc.
Original task card for this PR.