-
-
Notifications
You must be signed in to change notification settings - Fork 334
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] helpdesk_motive #526
Conversation
Co-Authored-By: Bhavesh Odedra <[email protected]>
Currently translated at 21.4% (3 of 14 strings) Translation: helpdesk-12.0/helpdesk-12.0-helpdesk_motive Translate-URL: https://translation.odoo-community.org/projects/helpdesk-12-0/helpdesk-12-0-helpdesk_motive/pt_BR/
Currently translated at 100.0% (14 of 14 strings) Translation: helpdesk-12.0/helpdesk-12.0-helpdesk_motive Translate-URL: https://translation.odoo-community.org/projects/helpdesk-12-0/helpdesk-12-0-helpdesk_motive/pt_BR/
Currently translated at 92.9% (13 of 14 strings) Translation: helpdesk-12.0/helpdesk-12.0-helpdesk_motive Translate-URL: https://translation.odoo-community.org/projects/helpdesk-12-0/helpdesk-12-0-helpdesk_motive/es/
Currently translated at 100.0% (14 of 14 strings) Translation: helpdesk-12.0/helpdesk-12.0-helpdesk_motive Translate-URL: https://translation.odoo-community.org/projects/helpdesk-12-0/helpdesk-12-0-helpdesk_motive/es/
Currently translated at 100.0% (14 of 14 strings) Translation: helpdesk-13.0/helpdesk-13.0-helpdesk_motive Translate-URL: https://translation.odoo-community.org/projects/helpdesk-13-0/helpdesk-13-0-helpdesk_motive/es_AR/
Maintainer Task
Currently translated at 100.0% (14 of 14 strings) Translation: helpdesk-13.0/helpdesk-13.0-helpdesk_motive Translate-URL: https://translation.odoo-community.org/projects/helpdesk-13-0/helpdesk-13-0-helpdesk_motive/es/
Currently translated at 100.0% (15 of 15 strings) Translation: helpdesk-14.0/helpdesk-14.0-helpdesk_motive Translate-URL: https://translation.odoo-community.org/projects/helpdesk-14-0/helpdesk-14-0-helpdesk_motive/it/
Currently translated at 100.0% (15 of 15 strings) Translation: helpdesk-14.0/helpdesk-14.0-helpdesk_motive Translate-URL: https://translation.odoo-community.org/projects/helpdesk-14-0/helpdesk-14-0-helpdesk_motive/fr/
Currently translated at 100.0% (14 of 14 strings) Translation: helpdesk-14.0/helpdesk-14.0-helpdesk_motive Translate-URL: https://translation.odoo-community.org/projects/helpdesk-14-0/helpdesk-14-0-helpdesk_motive/es_AR/
After OCA@d053593 `_onchange_dominion_user_id` has been removed in helpdesk_mgmt, so it cannot be extended anymore in helpdesk_motive.
Currently translated at 100.0% (15 of 15 strings) Translation: helpdesk-14.0/helpdesk-14.0-helpdesk_motive Translate-URL: https://translation.odoo-community.org/projects/helpdesk-14-0/helpdesk-14-0-helpdesk_motive/pt/
Currently translated at 100.0% (14 of 14 strings) Translation: helpdesk-14.0/helpdesk-14.0-helpdesk_motive Translate-URL: https://translation.odoo-community.org/projects/helpdesk-14-0/helpdesk-14-0-helpdesk_motive/es/
d26c32c
to
a059980
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.
Only code 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
This PR has the |
@nelsonramirezs @max3903 Can you merge this, please? |
LGTM |
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.
please rebase so that CI will be triggered and the merging process will continue
a059980
to
26035a6
Compare
26035a6
to
cdc6234
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.
Perform the tests and it is functional!
/ocabot migration helpdesk_motive |
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 9813927. Thanks a lot for contributing to OCA. ❤️ |
The api.onchange decorator has been changed to compute method for this change has been made to affect any changes in the user_id and team_id fields in the database, not only from the form.
Add test for compute method.