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
Master time off accrual plan crm #52298
Conversation
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.
First review.
Installing a linter is mandatory. Reading carefully our coding guidelines is strongly recommended.
A lot of cosmetic comments. But also some more technical. It would have been correct to define a o2m field instead of as many selection fields as possible to configure 4 periods.
Nothing about the way leaves are actually computed for the employees is already done. No unit tests too.
Still some polishing to make here + There's no test.
Thanks for your work anyway.
8a0f6e4
to
3a589cf
Compare
3a589cf
to
8311739
Compare
8311739
to
e0a09cf
Compare
e0a09cf
to
236d60e
Compare
236d60e
to
93bab2c
Compare
0fa522d
to
c9baf90
Compare
c9baf90
to
dc18138
Compare
c8124f2
to
ae77ccc
Compare
Hello @tivisse |
Add an accruapl plan model to the time off app
--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr