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

Order Number Not Being Passed For Internet Banking/Installments #214

Closed
BrainTurner opened this issue Jul 31, 2019 · 3 comments
Closed

Order Number Not Being Passed For Internet Banking/Installments #214

BrainTurner opened this issue Jul 31, 2019 · 3 comments
Assignees

Comments

@BrainTurner
Copy link

Environment:

PHP 7+
Magento 2.2.7 -> 2.3.2
Nginx
Varnish
Ubuntu 18.04
Custom Theme

Steps to reproduce:

  1. Add items to cart
  2. Click Checkout and login to site
  3. Enter Shipping/Billing info
  4. Select delivery options
  5. Choose internet banking or installment payemnt
  6. Click place order

Error:

GET http://********/en/rest/en/V1/orders//omise-offsite?_=1564125581958 400 (Bad Request)

Code causing the problem:

` function(response) {
var self = this;

                    var serviceUrl = urlBuilder.createUrl(
                        '/orders/:order_id/omise-offsite',
                        {
                            order_id: response
                        }
                    );

`
Expected Results:

Customer forwarded to payment gateway

This only happens with version omise-magento 2.8 amd didnt happen in previous versions. If i have a clean installation of magento i cannot reproduce the issue. This issue only happens with a custom theme.

@jacstn jacstn self-assigned this Sep 6, 2019
@jacstn
Copy link
Contributor

jacstn commented Sep 6, 2019

@BrainTurner still exists?
I also sent you private message some time ago.

@BrainTurner
Copy link
Author

Hi @jacstn
Sorry for late reply. I did manage to reproduce the issue on a clean install of magento. Its was due to minify HTML and java bundling. I disabled both and it worked in the end

@aashishgurung
Copy link
Contributor

We are closing this issue since it's already been resolved. Thank you.

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

No branches or pull requests

3 participants