-
-
Notifications
You must be signed in to change notification settings - Fork 675
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
[13.0][MIG] hr_job_category #895
Conversation
hr_job_category/models/hr.py
Outdated
|
||
def write(self, vals): | ||
res = super().write(vals) | ||
if "job_id" in vals: |
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.
if the employee was changed you are not removing tags from him
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.
Done!
05b8b66
to
93021fd
Compare
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.
LGTM
93021fd
to
339118f
Compare
339118f
to
a0ff71a
Compare
This PR has the |
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 51fa0af. Thanks a lot for contributing to OCA. ❤️ |
Migration of
hr_job_category
and simplified code.@etobella @Nikul-Chaudhary @i-vyshnevska @alexey-pelykh