Skip to content

[19.0][MIG] crm_phonecall: migration to 19.0#712

Open
eugenios73 wants to merge 76 commits intoOCA:19.0from
Studio73:19.0-mig-crm_phonecall
Open

[19.0][MIG] crm_phonecall: migration to 19.0#712
eugenios73 wants to merge 76 commits intoOCA:19.0from
Studio73:19.0-mig-crm_phonecall

Conversation

@eugenios73
Copy link

@eugenios73 eugenios73 commented Nov 28, 2025

migration crm_phonecall from 18.0 to 19.0

The pre-commit-config update in commit [19.0][FIX] update pre-commit-config is to resolve the "user_id" error issue that was shown in the last pre-commit command and which I commented on in this message #712 (comment)

@eugenios73 eugenios73 force-pushed the 19.0-mig-crm_phonecall branch 3 times, most recently from d0ec288 to b5cae86 Compare December 1, 2025 09:01
@eugenios73
Copy link
Author

The precommit command fails because it does not seem to be up to date with the change in v19.0 of the "user_ids" attribute of the base model "ir.filters". This attribute has changed compared to v18.0, where it was "user_id". Here is the commit where the change can be seen: odoo/odoo@414e55c

Copy link

@rrebollo rrebollo left a comment

Choose a reason for hiding this comment

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

Code Review: Great work! The code looks good to me (LGTM). Thank you for your contribution! I've provided a few suggestions for your consideration—feel free to address them as you see fit.

@rrebollo
Copy link

@eugenios73 could you review mine? #661

cubells and others added 24 commits January 15, 2026 15:20
* crm_phonecall module (OCA#131)

This module is the part extracted from Odoo v8 crm module from Odoo that allows to record phone calls. As it has been stripped off from Odoo v9, we add again the feature via this module.

Migration scripts are also included to upgrade a v8 database via OpenUpgrade and don't lose your history.

* [MIG] crm_phone_call: Migrated to 10.0

 - FIX: Correct some lint errors

* [MIG] crm_phone_call: Migrated to 10.0
Currently translated at 100.0% (150 of 150 strings)

Translation: crm-12.0/crm-12.0-crm_phonecall
Translate-URL: https://translation.odoo-community.org/projects/crm-12-0/crm-12-0-crm_phonecall/es/
Currently translated at 86.0% (129 of 150 strings)

Translation: crm-12.0/crm-12.0-crm_phonecall
Translate-URL: https://translation.odoo-community.org/projects/crm-12-0/crm-12-0-crm_phonecall/de/
Currently translated at 92.7% (139 of 150 strings)

Translation: crm-12.0/crm-12.0-crm_phonecall
Translate-URL: https://translation.odoo-community.org/projects/crm-12-0/crm-12-0-crm_phonecall/de/
Currently translated at 98.0% (147 of 150 strings)

Translation: crm-12.0/crm-12.0-crm_phonecall
Translate-URL: https://translation.odoo-community.org/projects/crm-12-0/crm-12-0-crm_phonecall/de/
Currently translated at 27.3% (41 of 150 strings)

Translation: crm-12.0/crm-12.0-crm_phonecall
Translate-URL: https://translation.odoo-community.org/projects/crm-12-0/crm-12-0-crm_phonecall/hr/
Currently translated at 41.3% (62 of 150 strings)

Translation: crm-12.0/crm-12.0-crm_phonecall
Translate-URL: https://translation.odoo-community.org/projects/crm-12-0/crm-12-0-crm_phonecall/hr/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: crm-13.0/crm-13.0-crm_phonecall
Translate-URL: https://translation.odoo-community.org/projects/crm-13-0/crm-13-0-crm_phonecall/
Antoni Marroig Campomar and others added 16 commits January 15, 2026 15:20
Currently translated at 100.0% (153 of 153 strings)

Translation: crm-17.0/crm-17.0-crm_phonecall
Translate-URL: https://translation.odoo-community.org/projects/crm-17-0/crm-17-0-crm_phonecall/it/
Currently translated at 100.0% (153 of 153 strings)

Translation: crm-17.0/crm-17.0-crm_phonecall
Translate-URL: https://translation.odoo-community.org/projects/crm-17-0/crm-17-0-crm_phonecall/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: crm-18.0/crm-18.0-crm_phonecall
Translate-URL: https://translation.odoo-community.org/projects/crm-18-0/crm-18-0-crm_phonecall/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: crm-18.0/crm-18.0-crm_phonecall
Translate-URL: https://translation.odoo-community.org/projects/crm-18-0/crm-18-0-crm_phonecall/
Currently translated at 99.3% (152 of 153 strings)

Translation: crm-18.0/crm-18.0-crm_phonecall
Translate-URL: https://translation.odoo-community.org/projects/crm-18-0/crm-18-0-crm_phonecall/es/
Currently translated at 99.3% (152 of 153 strings)

Translation: crm-18.0/crm-18.0-crm_phonecall
Translate-URL: https://translation.odoo-community.org/projects/crm-18-0/crm-18-0-crm_phonecall/es/
@eugenios73 eugenios73 force-pushed the 19.0-mig-crm_phonecall branch 2 times, most recently from ebca7da to d9fd486 Compare January 15, 2026 15:11
Copy link

@Reyes4711-S73 Reyes4711-S73 left a comment

Choose a reason for hiding this comment

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

LGTM

@eugenios73 eugenios73 force-pushed the 19.0-mig-crm_phonecall branch 3 times, most recently from 789e587 to f003208 Compare January 20, 2026 12:32
@eugenios73 eugenios73 force-pushed the 19.0-mig-crm_phonecall branch from f003208 to a6a52fe Compare January 20, 2026 13:05

Choose a reason for hiding this comment

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

I believe this type of contribution should be managed in a separate pull request. Then we can base this work on top of it.

Copy link
Author

Choose a reason for hiding this comment

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

@rrebollo new PR with the pre-commit-config changes #724

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@eugenios73 eugenios73 force-pushed the 19.0-mig-crm_phonecall branch 2 times, most recently from ffdb018 to 54ad4d8 Compare January 21, 2026 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.