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

[17.0][MIG] loyalty_partner_applicability: Migration to 17.0 #225

Open
wants to merge 14 commits into
base: 17.0
Choose a base branch
from

Conversation

arantxa-s73
Copy link

Module migrated to version 17.0

pilarvargas-tecnativa and others added 12 commits September 17, 2024 10:55
…me entity

There's a new configuration option in the settings to allow sharing
coupons between members of the same trading entity.

It has to be in the base module for being shared between all the rest of
applications, but each *_applicability module will contain extra bits of
code to use it.

TT45413
…main rule

It is necessary to establish the default value, in this case "[ ]" to the
rule_partners_domain field, in the create method for the rules established
in the computed method "_program_type_default_values" that establishes the
default values depending on the type of program, because when creating a
new program, these rules do not have the rule_partners_domain field defined,
as it is not contemplated, its value will be "False", which cannot be
interpreted as a correct domain. To avoid this error, the check is made in
the create method and the correct value is added.

TT44344
Currently translated at 100.0% (5 of 5 strings)

Translation: sale-promotion-16.0/sale-promotion-16.0-loyalty_partner_applicability
Translate-URL: https://translation.odoo-community.org/projects/sale-promotion-16-0/sale-promotion-16-0-loyalty_partner_applicability/it/
@arantxa-s73 arantxa-s73 changed the title [17.0][MIG] loyalty_partner_applicability: Migration to 16.0 [17.0][MIG] loyalty_partner_applicability: Migration to 17.0 Sep 17, 2024
@rousseldenis
Copy link

/ocabot migration loyalty_partner_applicability

@rousseldenis
Copy link

@arantxa-s73 It seems you miss migration commit (where at least version change should appear in) :

image

@arantxa-s73 arantxa-s73 force-pushed the 17.0-mig-loyalty_partner_applicability branch from 222a23b to 389f92b Compare September 26, 2024 08:11
@@ -2,7 +2,8 @@
# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
{
"name": "Loyalty Partner Applicability",
"summary": "Enables the definition of a customer filter for promotion rules that will "
"summary": "Enables the definition of a customer filter for promotion "
"rules that will "
"only be applied to customers who meet the specified conditions in the filter.",
"version": "16.0.2.0.2",
Copy link
Contributor

Choose a reason for hiding this comment

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

The version must be 17

Copy link
Author

Choose a reason for hiding this comment

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

Is modified in the following commit: [MIG] loyalty_partner_applicability: Migration to 17.0

Copy link
Contributor

Choose a reason for hiding this comment

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

Right, I was confused about commit, sorry

Can you push again to be able to run runbot?

Thanks!

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.

5 participants