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

[MIG] crm_lead_code: Migration to 18.0 #609

Open
wants to merge 31 commits into
base: 18.0
Choose a base branch
from

Conversation

HeliconiaSolutions
Copy link

No description provided.

Copy link

@bhaveshselarka bhaveshselarka left a comment

Choose a reason for hiding this comment

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

Functional Review, LGTM

Copy link

@CRogos CRogos left a comment

Choose a reason for hiding this comment

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

Could you cleanup the bot commits? I've marked some examples, but there are more.
image

I hope the the push will also recreate the runboat for testing

@HeliconiaSolutions HeliconiaSolutions force-pushed the 18.0-mig-crm_lead_code branch 2 times, most recently from f6d7aea to 6d4d297 Compare February 21, 2025 05:06
@HeliconiaSolutions
Copy link
Author

@CRogos , Please review.it's done!

@CRogos
Copy link

CRogos commented Feb 21, 2025

There are still more commits which can be merged.
Here is a link to the migration documentation: https://github.com/OCA/maintainer-tools/wiki/Merge-commits-in-pull-requests#mergesquash-the-commits-generated-by-bots-or-weblate

@HeliconiaSolutions
Copy link
Author

@CRogos, Please review!

@CRogos
Copy link

CRogos commented Feb 21, 2025

do you know why the tests are failing? Maybe we should make a rebase?

also there are still some weblate and oca-* commits.

Copy link

@CRogos CRogos 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 LGTM and it is working.

Not sure why the tests fail, and some Commits can still be squashed.

nicolasramos and others added 13 commits February 25, 2025 14:15
Currently translated at 100.0% (3 of 3 strings)

Translation: crm-10.0/crm-10.0-crm_lead_code
Translate-URL: https://translation.odoo-community.org/projects/crm-10-0/crm-10-0-crm_lead_code/de/
[UPD] Update crm_lead_code.pot
Currently translated at 100.0% (3 of 3 strings)

Translation: crm-12.0/crm-12.0-crm_lead_code
Translate-URL: https://translation.odoo-community.org/projects/crm-12-0/crm-12-0-crm_lead_code/pt_BR/
[UPD] Update crm_lead_code.pot

[UPD] README.rst
[UPD] Update crm_lead_code.pot
Currently translated at 100.0% (6 of 6 strings)

Translation: crm-14.0/crm-14.0-crm_lead_code
Translate-URL: https://translation.odoo-community.org/projects/crm-14-0/crm-14-0-crm_lead_code/es_AR/
Currently translated at 100.0% (3 of 3 strings)

Translation: crm-14.0/crm-14.0-crm_lead_code
Translate-URL: https://translation.odoo-community.org/projects/crm-14-0/crm-14-0-crm_lead_code/es/
ajaniszewska-dev and others added 13 commits February 25, 2025 14:15
Currently translated at 100.0% (3 of 3 strings)

Translation: crm-15.0/crm-15.0-crm_lead_code
Translate-URL: https://translation.odoo-community.org/projects/crm-15-0/crm-15-0-crm_lead_code/fr/
Currently translated at 100.0% (3 of 3 strings)

Translation: crm-15.0/crm-15.0-crm_lead_code
Translate-URL: https://translation.odoo-community.org/projects/crm-15-0/crm-15-0-crm_lead_code/ca/
Currently translated at 66.6% (2 of 3 strings)

Translation: crm-15.0/crm-15.0-crm_lead_code
Translate-URL: https://translation.odoo-community.org/projects/crm-15-0/crm-15-0-crm_lead_code/it/
Currently translated at 25.0% (1 of 4 strings)

Translation: crm-16.0/crm-16.0-crm_lead_code
Translate-URL: https://translation.odoo-community.org/projects/crm-16-0/crm-16-0-crm_lead_code/sl/
Currently translated at 100.0% (4 of 4 strings)

Translation: crm-16.0/crm-16.0-crm_lead_code
Translate-URL: https://translation.odoo-community.org/projects/crm-16-0/crm-16-0-crm_lead_code/it/
Currently translated at 100.0% (4 of 4 strings)

Translation: crm-16.0/crm-16.0-crm_lead_code
Translate-URL: https://translation.odoo-community.org/projects/crm-16-0/crm-16-0-crm_lead_code/es/
Currently translated at 100.0% (4 of 4 strings)

Translation: crm-16.0/crm-16.0-crm_lead_code
Translate-URL: https://translation.odoo-community.org/projects/crm-16-0/crm-16-0-crm_lead_code/es_AR/
Currently translated at 75.0% (3 of 4 strings)

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

Translation: crm-17.0/crm-17.0-crm_lead_code
Translate-URL: https://translation.odoo-community.org/projects/crm-17-0/crm-17-0-crm_lead_code/pt_BR/
@HeliconiaSolutions
Copy link
Author

Hello @CRogos Commits are squashed and Tests are also clear now.

@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). 🤖

Copy link

@CRogos CRogos left a comment

Choose a reason for hiding this comment

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

Sorry, just found one more thing.

("crm_lead_unique_code", "UNIQUE (code)", _("The code must be unique!")),
("crm_lead_unique_code", "UNIQUE (code)", "The code must be unique!"),
Copy link

Choose a reason for hiding this comment

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

This change is not correct.
You should replace _ by self.env._ or assign _ = self.env._

https://www.odoo.com/documentation/18.0/contributing/development/coding_guidelines.html#use-translation-method-correctly

Or was it intended to remove the translation?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.