Skip to content
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

Merged
merged 28 commits into from
May 31, 2024
Merged

Conversation

adasatorres
Copy link

Change onchange per compute, and created tests.

@adasatorres adasatorres changed the title 15.0 mig helpdesk motive [15.0][MIG]helpdesk motive Dec 22, 2023
@adasatorres adasatorres force-pushed the 15.0-mig-helpdesk_motive branch from ed01c61 to b96ba93 Compare December 22, 2023 09:03
Copy link
Contributor

@moitabenfdz moitabenfdz left a 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

Copy link

github-actions bot commented May 5, 2024

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.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label May 5, 2024
Maxime Chambreuil and others added 23 commits May 31, 2024 14:39
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/
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/
@adasatorres adasatorres force-pushed the 15.0-mig-helpdesk_motive branch from b96ba93 to 65bb921 Compare May 31, 2024 13:39
@adasatorres adasatorres requested a review from moitabenfdz May 31, 2024 13:53
Copy link
Contributor

@Christian-RB Christian-RB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@marcelsavegnago marcelsavegnago requested review from max3903 and nelsonramirezs and removed request for moitabenfdz May 31, 2024 14:15
@max3903 max3903 added this to the 15.0 milestone May 31, 2024
@max3903
Copy link
Member

max3903 commented May 31, 2024

/ocabot migration helpdesk_motive

@OCA-git-bot OCA-git-bot mentioned this pull request May 31, 2024
8 tasks
@max3903
Copy link
Member

max3903 commented May 31, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-527-by-max3903-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 9f3f28c into OCA:15.0 May 31, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 9b2ecd3. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged 🎉 migration needs review stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Development

Successfully merging this pull request may close these issues.