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] marketing_crm_partner: Migration to version 15.0 #504

Merged
merged 24 commits into from
Jul 4, 2023

Conversation

stefan-tecnativa
Copy link
Contributor

cc @Tecnativa TT44157

@chienandalu please review!

yajo and others added 2 commits July 3, 2023 15:15
This module extends the functionality of the CRM to support having the tracking
fields available in the partner and copy them there automatically when the
partner is created from a lead/opportunity.
Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

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

Please squash some administrative commits

cubells and others added 21 commits July 4, 2023 07:50
Currently translated at 100.0% (9 of 9 strings)

Translation: crm-11.0/crm-11.0-marketing_crm_partner
Translate-URL: https://translation.odoo-community.org/projects/crm-11-0/crm-11-0-marketing_crm_partner/de/
Currently translated at 100.0% (9 of 9 strings)

Translation: crm-11.0/crm-11.0-marketing_crm_partner
Translate-URL: https://translation.odoo-community.org/projects/crm-11-0/crm-11-0-marketing_crm_partner/gl/
Currently translated at 100.0% (6 of 6 strings)

Translation: crm-12.0/crm-12.0-marketing_crm_partner
Translate-URL: https://translation.odoo-community.org/projects/crm-12-0/crm-12-0-marketing_crm_partner/pt_BR/
Since odoo/odoo@e183393, we need to reassign the partner to the lead, as the
method doesn't do it by itself.
Currently translated at 100.0% (6 of 6 strings)

Translation: crm-12.0/crm-12.0-marketing_crm_partner
Translate-URL: https://translation.odoo-community.org/projects/crm-12-0/crm-12-0-marketing_crm_partner/hr/
Currently translated at 100.0% (6 of 6 strings)

Translation: crm-12.0/crm-12.0-marketing_crm_partner
Translate-URL: https://translation.odoo-community.org/projects/crm-12-0/crm-12-0-marketing_crm_partner/ca/
Currently translated at 100.0% (6 of 6 strings)

Translation: crm-12.0/crm-12.0-marketing_crm_partner
Translate-URL: https://translation.odoo-community.org/projects/crm-12-0/crm-12-0-marketing_crm_partner/es/
Currently translated at 83.3% (5 of 6 strings)

Translation: crm-12.0/crm-12.0-marketing_crm_partner
Translate-URL: https://translation.odoo-community.org/projects/crm-12-0/crm-12-0-marketing_crm_partner/it/
Currently translated at 100.0% (6 of 6 strings)

Translation: crm-12.0/crm-12.0-marketing_crm_partner
Translate-URL: https://translation.odoo-community.org/projects/crm-12-0/crm-12-0-marketing_crm_partner/sl/
Currently translated at 100.0% (6 of 6 strings)

Translation: crm-13.0/crm-13.0-marketing_crm_partner
Translate-URL: https://translation.odoo-community.org/projects/crm-13-0/crm-13-0-marketing_crm_partner/es_AR/
Currently translated at 100.0% (6 of 6 strings)

Translation: crm-14.0/crm-14.0-marketing_crm_partner
Translate-URL: https://translation.odoo-community.org/projects/crm-14-0/crm-14-0-marketing_crm_partner/it/
@stefan-tecnativa
Copy link
Contributor Author

Please squash some administrative commits

Done, thanks for the comment

Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

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

Please review @CarlosRoca13

Copy link

@CarlosRoca13 CarlosRoca13 left a comment

Choose a reason for hiding this comment

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

Minimal comment


#. Go to **CRM > Leads > Create**.
#. Fill the required fields.
#. Go to **Extra Info > Marketing** and fill those 3 fields.

Choose a reason for hiding this comment

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

Suggested change
#. Go to **Extra Info > Marketing** and fill those 3 fields.
#. Go to **Extra Info > Tracking** and fill those 3 fields.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done, thanks!

@pedrobaeza
Copy link
Member

/ocabot merge nobump
/ocabot migration marketing_crm_partner

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Jul 4, 2023
@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot mentioned this pull request Jul 4, 2023
16 tasks
OCA-git-bot added a commit that referenced this pull request Jul 4, 2023
Signed-off-by pedrobaeza
@OCA-git-bot
Copy link
Contributor

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-504-by-pedrobaeza-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 15.0-ocabot-merge-pr-504-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit ee6edbe into OCA:15.0 Jul 4, 2023
@OCA-git-bot
Copy link
Contributor

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

@francescapenso
Copy link

This should be migrate to 16.0?

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.