-
-
Notifications
You must be signed in to change notification settings - Fork 664
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
[16.0][OU-ADD] loyalty #4187
base: 16.0
Are you sure you want to change the base?
[16.0][OU-ADD] loyalty #4187
Conversation
/ocabot migration loyalty |
fcfcc65
to
e171bdd
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.
Starting to look quite good :)
- Check CIs
- Comment code as much as you can
openupgrade_scripts/scripts/loyalty/16.0.1.0/noupdate_changes.xml
Outdated
Show resolved
Hide resolved
b7584b1
to
824798f
Compare
824798f
to
735c874
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.
Please, check api call
735c874
to
5912a50
Compare
openupgrade.rename_xmlids(env.cr, _xmlids_renames) | ||
# Renamed in rename_xmlids method. | ||
# In v15 are noupdate=1 and in v16 are noupdate=0 | ||
openupgrade.set_xml_ids_noupdate_value( |
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.
The error message when testing the upgrade script is as follows:
odoo.exceptions.UserError: XML IDs mail_template_loyalty_card must be a tuple or list!
These lines of code can be changed to:
openupgrade.set_xml_ids_noupdate_value(
env, "loyalty", ["mail_template_loyalty_card"], False
)
openupgrade.set_xml_ids_noupdate_value(
env, "loyalty", ["gift_card_product_50"], False
)
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.
thanks
5912a50
to
040d8e9
Compare
TT44332
040d8e9
to
57af6c1
Compare
cc @Tecnativa TT44332
@chienandalu @ernesto-garcia-tecnativa please review
ping @pedrobaeza