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

Jump to bottom

Order id undefined #119

Closed
rotexhawk opened this issue Aug 4, 2019 · 1 comment
Closed

Order id undefined #119

rotexhawk opened this issue Aug 4, 2019 · 1 comment
Labels
bug Something isn't working design notes Blueprint and notes for coming feature failing mutation Issue concerns a failing/broken mutation
Milestone
v0.2.1

Comments

@rotexhawk
Copy link

I am trying to debug why the checkout mutation succeeds without actually creating an order in wordpress. I enabled the logs and wordpress complains about $order_id on the very last function in includes/data/mutation/class-checkout-mutation.php. I guess if the condition fails the order_id is undeclared.
@rotexhawk
Copy link
Author

also the condition to create an order has to go through this.
if ( empty( $data['woocommerce_checkout_update_totals'] ) ) { so passing updateTotals:true in graphql will never create an order. What is the purpose of updateTotals?

@kidunot89 kidunot89 self-assigned this Aug 5, 2019
@kidunot89 kidunot89 added bug Something isn't working design notes Blueprint and notes for coming feature failing mutation Issue concerns a failing/broken mutation labels Aug 5, 2019
@kidunot89 kidunot89 added this to the v0.2.1 milestone Aug 5, 2019
@kidunot89 kidunot89 removed their assignment Aug 5, 2019
@kidunot89 kidunot89 mentioned this issue Aug 18, 2019
Merged
15 tasks
@kidunot89 kidunot89 mentioned this issue Aug 26, 2019
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working design notes Blueprint and notes for coming feature failing mutation Issue concerns a failing/broken mutation
Projects
None yet
Milestone
v0.2.1
Development

No branches or pull requests
2 participants
@rotexhawk @kidunot89