-
-
Notifications
You must be signed in to change notification settings - Fork 673
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
[16.0][MIG] hr_contract_employee_calendar_planning #1324
[16.0][MIG] hr_contract_employee_calendar_planning #1324
Conversation
Currently translated at 100.0% (1 of 1 strings) Translation: hr-15.0/hr-15.0-hr_contract_employee_calendar_planning Translate-URL: https://translation.odoo-community.org/projects/hr-15-0/hr-15-0-hr_contract_employee_calendar_planning/it/
…ement - Include context keys for avoiding mail operations overhead.
…ite from contract creation
…ndars chronologically
…creating contracts with different work hours
e844150
to
8181060
Compare
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
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.
Tested locally and it looks good to me! However, in the migration proccess code has been added, such as overwitten create method and modifications to hr.contract.history view, in addition to the already modified hr.contrat view. The overwritten create method should be tested in order to pass all checks. Can you add that test?
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
8181060
to
591e1a3
Compare
/ocabot migration hr_contract_employee_calendar_planning |
591e1a3
to
da038b2
Compare
@pedrobaeza |
I don't use this module. @etobella should say. And each PR requires 2 approvals, and one from PSC: https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#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.
/ocabot merge nobump
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at e5cc480. Thanks a lot for contributing to OCA. ❤️ |
Migration to 16.0