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

BhaveshHeliconia
Copy link

No description provided.

Copy link

@patel1008b patel1008b 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

@BhaveshHeliconia BhaveshHeliconia force-pushed the 18.0-mig-crm_lead_code branch 2 times, most recently from f6d7aea to 6d4d297 Compare February 21, 2025 05:06
@BhaveshHeliconia
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

@BhaveshHeliconia
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/
@BhaveshHeliconia
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.

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

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.