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.

Sign up for GitHub

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] website_sale_loyalty_delivery: Migration to version 16.0 #4527

Merged
merged 6 commits into from
Jul 20, 2024
Merged

[16.0][OU-ADD] website_sale_loyalty_delivery: Migration to version 16.0 #4527

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
6447b86
[OU-ADD] website_sale_loyalty_delivery: Migration to version 16.0
pilarvargas-tecnativa Jul 17, 2024
c8ed837
[OU-IMP] sale_loyalty_delivery: Adding information to the module cove…
pilarvargas-tecnativa Jul 18, 2024
8dc6baf
[OU-IMP] loyalty: Adding information to the module coverage table
pilarvargas-tecnativa Jul 18, 2024
f8eb983
[OU-IMP] sale_loyalty: Adding information to the module coverage table
pilarvargas-tecnativa Jul 18, 2024
2262c37
[OU-IMP] website_sale_loyalty: Adding information to the module cover…
pilarvargas-tecnativa Jul 18, 2024
ff2ee1c
[OU-FIX] sale_loyalty: Remove script that removes constraint
pilarvargas-tecnativa Jul 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 10 additions & 10 deletions docsource/modules150-160.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ Module coverage 15.0 -> 16.0
+-------------------------------------------------+----------------------+-------------------------------------------------+
| contacts | Nothing to do |No DB layout changes. |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| |del| coupon | | |
| |del| coupon | Done |Renamed to loyalty |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| crm | Done | |
+-------------------------------------------------+----------------------+-------------------------------------------------+
Expand Down Expand Up @@ -454,7 +454,7 @@ Module coverage 15.0 -> 16.0
+-------------------------------------------------+----------------------+-------------------------------------------------+
| link_tracker | Nothing to do |No DB layout changes. |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| |new| loyalty | | |
| |new| loyalty | Done |Renamed from coupon |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| |new| loyalty_delivery | | |
+-------------------------------------------------+----------------------+-------------------------------------------------+
Expand Down Expand Up @@ -680,9 +680,9 @@ Module coverage 15.0 -> 16.0
+-------------------------------------------------+----------------------+-------------------------------------------------+
| sale | Done | |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| |del| sale_coupon | | |
| |del| sale_coupon | Done |Renamed to sale_loyalty |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| |del| sale_coupon_delivery | | |
| |del| sale_coupon_delivery | Done |Renamed to sale_loyalty_delivery |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| sale_crm | Nothing to do | |
+-------------------------------------------------+----------------------+-------------------------------------------------+
Expand All @@ -692,9 +692,9 @@ Module coverage 15.0 -> 16.0
+-------------------------------------------------+----------------------+-------------------------------------------------+
| |del| sale_gift_card | | |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| |new| sale_loyalty | | |
| |new| sale_loyalty | Done |Renamed from sale_coupon |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| |new| sale_loyalty_delivery | | |
| |new| sale_loyalty_delivery | Done |Renamed from sale_coupon_delivery |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| sale_management | Done | |
+-------------------------------------------------+----------------------+-------------------------------------------------+
Expand Down Expand Up @@ -892,9 +892,9 @@ Module coverage 15.0 -> 16.0
+-------------------------------------------------+----------------------+-------------------------------------------------+
| website_sale_comparison_wishlist | | |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| |del| website_sale_coupon | | |
| |del| website_sale_coupon | Done |Renamed to website_sale_loyalty |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| |del| website_sale_coupon_delivery | | |
| |del| website_sale_coupon_delivery | Done |Renamed to website_sale_loyalty_delivery |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| website_sale_delivery | Nothing to do |No DB layout changes. |
+-------------------------------------------------+----------------------+-------------------------------------------------+
Expand All @@ -906,9 +906,9 @@ Module coverage 15.0 -> 16.0
+-------------------------------------------------+----------------------+-------------------------------------------------+
| |del| website_sale_gift_card | | |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| |new| website_sale_loyalty | | |
| |new| website_sale_loyalty | Done |Renamed from website_sale_coupon |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| |new| website_sale_loyalty_delivery | | |
| |new| website_sale_loyalty_delivery | Done |Renamed from website_sale_coupon_delivery |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| |new| website_sale_picking | | |
+-------------------------------------------------+----------------------+-------------------------------------------------+
Expand Down
22 changes: 13 additions & 9 deletions openupgrade_scripts/scripts/sale_loyalty/16.0.1.0/post-migration.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -80,15 +80,19 @@ def _generate_sale_order_coupon_points(env):
coupon_id, order_id, points, create_uid, write_uid, create_date, write_date
)
SELECT
id AS coupon_id,
order_id,
points,
create_uid,
write_uid,
create_date,
write_date
FROM loyalty_card
WHERE order_id IS NOT NULL;
lc.id AS coupon_id,
lc.order_id,
lc.points,
lc.create_uid,
lc.write_uid,
lc.create_date,
lc.write_date
FROM loyalty_card lc
LEFT JOIN sale_order_coupon_points socp
ON lc.order_id = socp.order_id
AND lc.id = socp.coupon_id
WHERE lc.order_id IS NOT NULL
AND socp.id IS NULL
""",
)

Expand Down
8 changes: 0 additions & 8 deletions openupgrade_scripts/scripts/sale_loyalty/16.0.1.0/pre-migration.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -138,13 +138,6 @@ def update_sale_order_line_data(env):
)


def delete_sql_constraints(env):
# Delete constraints to recreate it
openupgrade.delete_sql_constraint_safely(
env, "sale_loyalty", "sale_order_coupon_points", "order_coupon_unique"
)


def update_template_keys(env):
"""Update template keys of the merged sale_gift_card module in loyalty_sale"""
openupgrade.logged_query(
Expand Down Expand Up @@ -178,5 +171,4 @@ def migrate(env, version):
openupgrade.rename_xmlids(env.cr, _xmlids_renames)
update_loyalty_program_data(env)
update_sale_order_line_data(env)
delete_sql_constraints(env)
update_template_keys(env)
2 changes: 1 addition & 1 deletion openupgrade_scripts/scripts/sale_loyalty/16.0.1.0/upgrade_analysis_work.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ DEL ir.model.access: sale_gift_card.access_gift_card_sales
# NOTHING TO DO

NEW ir.model.constraint: sale_loyalty.constraint_sale_order_coupon_points_order_coupon_unique
# DONE pre-migration: safely delete constraint to recreate it
# NOTHING TO DO

DEL ir.rule: sale_coupon.sale_coupon_apply_code_rule (noupdate)
# DONE: post-migration: safely deleted xmlid
Expand Down
Loading