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] website_sale_cart_expire: Migration #923

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

Conversation

Mantux11
Copy link
Contributor

Module website_sale_cart_expire migration to 17.0.

ivantodorovich and others added 26 commits May 10, 2024 14:46
Module that allows to automatically cancel carts without
activity after a configurable time.
In a typical website flow, quotations are moved to sent state when
a payment by Wire transfer is chosen.

In this scenario, the quotation shouldn't be automatically cancelled.

It's up to the salesman to determine when a sent quotation is expired,
as technically speaking this isn't just a cart anymore.
[Fix] Use new payment.payment_provider_demo for test cases

[Fix] Change acquirer_id to provider_id for test cases
Currently translated at 100.0% (14 of 14 strings)

Translation: e-commerce-16.0/e-commerce-16.0-website_sale_cart_expire
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-16-0/e-commerce-16-0-website_sale_cart_expire/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: e-commerce-16.0/e-commerce-16.0-website_sale_cart_expire
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-16-0/e-commerce-16-0-website_sale_cart_expire/
Before this commit, there are an issue the method raise
an error if transaction_ids.last_state_change was false.
It's possible for last_state_change to be false as a result
of migrating data from a past Odoo DB version.
Currently translated at 100.0% (14 of 14 strings)

Translation: e-commerce-16.0/e-commerce-16.0-website_sale_cart_expire
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-16-0/e-commerce-16-0-website_sale_cart_expire/it/
@Mantux11 Mantux11 mentioned this pull request May 10, 2024
27 tasks
@Mantux11 Mantux11 marked this pull request as draft May 10, 2024 14:04
@Mantux11 Mantux11 force-pushed the 17.0-27851-website_sale_cart_expire branch from a46663a to dd27f68 Compare May 10, 2024 14:22
@Mantux11 Mantux11 marked this pull request as ready for review May 10, 2024 14:23
Copy link

@DorianMAG DorianMAG left a comment

Choose a reason for hiding this comment

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

Hi @Mantux11
I m not sure to understand why, we find a binary file (.deb) in the PR
https://github.com/OCA/e-commerce/pull/923/files#diff-de4e69c46f94e742c70468b9e0f08ebc21bf93aa8a5b98463faf63effa1e7c8a

Same for the migration v15 folder?

Functional test it's ok

@Mantux11
Copy link
Contributor Author

Mantux11 commented Jun 26, 2024

@DorianMAG Hey. I don't know why there is a random file. As I see, pre-commit added or updated it? [IMP] website_sale_cart_expire: pre-commit auto fixes.

Personally, i don't know what this file should do and why it was added in the first place.
And by deleting it I guess pre-commit check would be red.

@pedrobaeza
Copy link
Member

It must be removed in the same commit that added it.

@Mantux11 Mantux11 force-pushed the 17.0-27851-website_sale_cart_expire branch from dd27f68 to 42bb648 Compare June 28, 2024 12:24
@Mantux11
Copy link
Contributor Author

@pedrobaeza file removed.

@pedrobaeza
Copy link
Member

Thanks.

/ocabot migration website_sale_cart_expire

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Jun 28, 2024
Copy link

@DorianMAG DorianMAG left a comment

Choose a reason for hiding this comment

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

Thx for this work

LGTM

Copy link
Member

@gaikaz gaikaz left a comment

Choose a reason for hiding this comment

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

This is already used in production since the creation of the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.