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_checkout_skip_payment #962

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

Conversation

alexeirivera87
Copy link

No description provided.

@alexeirivera87 alexeirivera87 force-pushed the 17.0-mig-website_sale_checkout_skip_payment branch 2 times, most recently from da1d206 to 1560075 Compare August 6, 2024 02:53
@pedrobaeza pedrobaeza changed the title [17.0][MIG] website sale checkout skip payment [17.0][MIG] website_sale_checkout_skip_payment Aug 6, 2024
@pedrobaeza
Copy link
Member

#939 already migrates this module, but both have red CI

@alexeirivera87 alexeirivera87 force-pushed the 17.0-mig-website_sale_checkout_skip_payment branch 2 times, most recently from 3fc3566 to 18c5d1e Compare September 10, 2024 15:52
@alexeirivera87
Copy link
Author

Hello @pedrobaeza @sergio-teruel

I fixed most of the tests, but still one raises when running pre-commit:

Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

It is related to this line:

import {registry} from "@web/core/registry";

Can you help me to fix it?

regards

@alexeirivera87
Copy link
Author

Hello @chienandalu

Any news?

regards

@pilarvargas-tecnativa
Copy link
Contributor

@alexeirivera87 To avoid the pre-commit error you must change the .js file extension to .esm.js.

@@ -1,47 +1,26 @@
odoo.define("website_sale_checkout_skip_payment.tour", function (require) {
"use strict";
/** @odoo-module */
Copy link
Contributor

Choose a reason for hiding this comment

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

/** @odoo-module **/

@alexeirivera87 alexeirivera87 force-pushed the 17.0-mig-website_sale_checkout_skip_payment branch from 18c5d1e to 09eca7a Compare November 12, 2024 18:25
sergio-teruel and others added 18 commits November 12, 2024 13:33
OCA Transbot updated translations from Transifex
…ut confirmation step when partner is has skip payment flag
Currently translated at 25.0% (2 of 8 strings)

Translation: e-commerce-11.0/e-commerce-11.0-website_sale_checkout_skip_payment
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-11-0/e-commerce-11-0-website_sale_checkout_skip_payment/it/
Currently translated at 87.5% (7 of 8 strings)

Translation: e-commerce-11.0/e-commerce-11.0-website_sale_checkout_skip_payment
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-11-0/e-commerce-11-0-website_sale_checkout_skip_payment/es/
Currently translated at 100.0% (8 of 8 strings)

Translation: e-commerce-11.0/e-commerce-11.0-website_sale_checkout_skip_payment
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-11-0/e-commerce-11-0-website_sale_checkout_skip_payment/es/
Currently translated at 33.3% (5 of 15 strings)

Translation: e-commerce-13.0/e-commerce-13.0-website_sale_checkout_skip_payment
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-13-0/e-commerce-13-0-website_sale_checkout_skip_payment/nl/
Currently translated at 100.0% (15 of 15 strings)

Translation: e-commerce-13.0/e-commerce-13.0-website_sale_checkout_skip_payment
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-13-0/e-commerce-13-0-website_sale_checkout_skip_payment/ca/
Currently translated at 100.0% (15 of 15 strings)

Translation: e-commerce-13.0/e-commerce-13.0-website_sale_checkout_skip_payment
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-13-0/e-commerce-13-0-website_sale_checkout_skip_payment/es/
…ttons

With this changes the 'Process Checkout' button change to 'Confirm' when the partner has selected the option skip_website_checkout_payment. Furthermore, the button text 'Confirm Order' has been changed to 'Confirm'.

TT29736
… user

When the client is a public user we got an error, because the request.session.uid is None and the value of the field checkout_skip_payment is not computed.

As we want to compute this field always,the condition is removed to avoid the possible error.

TT29736
OCA-git-bot and others added 27 commits November 12, 2024 13:33
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: e-commerce-16.0/e-commerce-16.0-website_sale_checkout_skip_payment
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-16-0/e-commerce-16-0-website_sale_checkout_skip_payment/
Currently translated at 100.0% (16 of 16 strings)

Translation: e-commerce-16.0/e-commerce-16.0-website_sale_checkout_skip_payment
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-16-0/e-commerce-16-0-website_sale_checkout_skip_payment/es_AR/
Currently translated at 100.0% (16 of 16 strings)

Translation: e-commerce-16.0/e-commerce-16.0-website_sale_checkout_skip_payment
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-16-0/e-commerce-16-0-website_sale_checkout_skip_payment/es/
VAT is required by website_sale_vat_required module

TT44388
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: e-commerce-16.0/e-commerce-16.0-website_sale_checkout_skip_payment
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-16-0/e-commerce-16-0-website_sale_checkout_skip_payment/
- Include context keys for avoiding mail operations overhead.
Currently translated at 100.0% (16 of 16 strings)

Translation: e-commerce-16.0/e-commerce-16.0-website_sale_checkout_skip_payment
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-16-0/e-commerce-16-0-website_sale_checkout_skip_payment/fr/
Currently translated at 100.0% (16 of 16 strings)

Translation: e-commerce-16.0/e-commerce-16.0-website_sale_checkout_skip_payment
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-16-0/e-commerce-16-0-website_sale_checkout_skip_payment/it/
Currently translated at 100.0% (16 of 16 strings)

Translation: e-commerce-16.0/e-commerce-16.0-website_sale_checkout_skip_payment
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-16-0/e-commerce-16-0-website_sale_checkout_skip_payment/pt_BR/
Currently translated at 100.0% (16 of 16 strings)

Translation: e-commerce-16.0/e-commerce-16.0-website_sale_checkout_skip_payment
Translate-URL: https://translation.odoo-community.org/projects/e-commerce-16-0/e-commerce-16-0-website_sale_checkout_skip_payment/sv/
When an order is confirmed and goes to the path ‘/shop/confirmation’,
the driver method involved is ‘shop_payment_confirmation’ from websitesale.
With the migration, the wrong method was involved as it was a portal
method and its path is ‘/payment/confirmation’, which has nothing to
do with the confirmation of an order.

TT50360
@alexeirivera87 alexeirivera87 force-pushed the 17.0-mig-website_sale_checkout_skip_payment branch from 09eca7a to 768e629 Compare November 12, 2024 18:34
@alexeirivera87
Copy link
Author

Thanks a lot @pilarvargas-tecnativa

regards

@alexeirivera87
Copy link
Author

Let me know @chienandalu

regards

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

Successfully merging this pull request may close these issues.