-
-
Notifications
You must be signed in to change notification settings - Fork 411
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
base: 18.0
Are you sure you want to change the base?
Conversation
8bbb507
to
427245d
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.
Functional Review, LGTM
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.
f6d7aea
to
6d4d297
Compare
@CRogos , Please review.it's done! |
There are still more commits which can be merged. |
6d4d297
to
8fe6ace
Compare
@CRogos, Please review! |
do you know why the tests are failing? Maybe we should make a rebase? also there are still some weblate and oca-* commits. |
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.
Code review LGTM and it is working.
Not sure why the tests fail, and some Commits can still be squashed.
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/
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/
8fe6ace
to
9193810
Compare
9193810
to
b9c09ab
Compare
Hello @CRogos Commits are squashed and Tests are also clear now. |
This PR has the |
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.
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!"), |
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.
This change is not correct.
You should replace _ by self.env._ or assign _ = self.env._
Or was it intended to remove the translation?
No description provided.