-
-
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
[15.0][MIG]helpdesk motive #527
Conversation
ed01c61
to
b96ba93
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.
The code LGTM and it works, but you should squash the administrative commits.
Check this -> https://github.com/OCA/maintainer-tools/wiki/Merge-commits-in-pull-requests#mergesquash-the-commits-generated-by-bots-or-weblate
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. |
Co-Authored-By: Bhavesh Odedra <bodedra@opensourceintegrators.com>
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/
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/
b96ba93
to
65bb921
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
/ocabot migration helpdesk_motive |
/ocabot merge nobump |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at 9b2ecd3. Thanks a lot for contributing to OCA. ❤️ |
Change onchange per compute, and created tests.