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

magento/devdocs#: Add info about setPaymentMethodAndPlaceOrder to GraphQL checkout tutorial #4950

Conversation

atwixfirster
Copy link
Contributor

@atwixfirster atwixfirster commented Jul 14, 2019

Purpose of this pull request

This pull request (PR) adds information about setPaymentMethodAndPlaceOrder mutation and provides additional changes for "Step 4. Set the shipping address", "Step 5. Set billing address" "Step 6. Set the shipping method" pages.

Affected DevDocs pages

Links to Magento source code

Thank you!

CC: @keharper

whatsnew
Updated the GraphQL checkout tutorial to include the setPaymentMethodAndPlaceOrder mutation.

@devops-devdocs
Copy link
Collaborator

An admin must run tests on this PR before it can be merged.

@rogyar rogyar self-assigned this Jul 15, 2019
@@ -96,6 +106,44 @@ If the operation is successful, the response contains the code of the selected p
}
```

### Set payment method and place order {#setPaymentMethodAndPlaceOrder}

Use the `setPaymentMethodAndPlaceOrder` mutation to set the payment method for your order.
Copy link
Contributor

Choose a reason for hiding this comment

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

Please, add a clarification here

Suggested change
Use the `setPaymentMethodAndPlaceOrder` mutation to set the payment method for your order.
Use the `setPaymentMethodAndPlaceOrder` mutation to set the payment method and place the order.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good point @rogyar

fixed

@rogyar rogyar added 2.3.x Magento 2.3 related changes Major Update Significant original updates to existing content labels Jul 15, 2019
@keharper keharper self-assigned this Jul 16, 2019
@@ -96,6 +106,44 @@ If the operation is successful, the response contains the code of the selected p
}
```

### Set payment method and place order {#setPaymentMethodAndPlaceOrder}
Copy link
Contributor

Choose a reason for hiding this comment

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

I've just realized that this mutation is not a part of 2.3.2 version. I would suggest splitting this PR into two separate PRs and making the PR with information about setPaymentMethodAndPlaceOrder mutation to the develop branch.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've created a separate PR #5000 to fix

  1. Remove address_type from shipping_addresses and billing_addresses.
  2. Remove label from selected_shipping_method

Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you. The only change left is to create a PR to the develop branch.

Copy link
Contributor

Choose a reason for hiding this comment

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

@rogyar I will reset the target branch for this PR into the develop branch immediately after the next time we sync master and develop.

@rogyar rogyar added the Waiting for Response Waiting for response from internal/external parties label Jul 19, 2019
@keharper keharper changed the base branch from master to develop July 22, 2019 14:23
@keharper keharper added xx2.3.3 Magento 2.3.3 changes and removed 2.3.x Magento 2.3 related changes labels Jul 25, 2019
Copy link
Contributor

@keharper keharper left a comment

Choose a reason for hiding this comment

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

@atwixfirster Just one technical question remains, then I can merge the PR.


**Response**

If the operation is successful, the response contains the code of the selected payment method.
Copy link
Contributor

Choose a reason for hiding this comment

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

Isn't this the order ID?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

order ID, @keharper

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@keharper keharper added Community Docs impacted by community code contribution and removed Waiting for Response Waiting for response from internal/external parties labels Aug 18, 2019
@keharper
Copy link
Contributor

running tests

Copy link
Contributor

@keharper keharper left a comment

Choose a reason for hiding this comment

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

There's a bad link.

|Mutation|Description|
|--- |--- |
|`setPaymentMethodOnCart`|Sets the payment method for your order|
|`setPaymentMethodAndPlaceOrder`|Sets the payment method and then immediately places your order. In this case ["Step 10. Place the order"]({{ page.baseurl }}/graphql/tutorials/checkout/checkout-place-order.md) can be skipped|
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
|`setPaymentMethodAndPlaceOrder`|Sets the payment method and then immediately places your order. In this case ["Step 10. Place the order"]({{ page.baseurl }}/graphql/tutorials/checkout/checkout-place-order.md) can be skipped|
|`setPaymentMethodAndPlaceOrder`|Sets the payment method and then immediately places your order. In this case ["Step 10. Place the order"]({{ page.baseurl }}/graphql/tutorials/checkout/checkout-place-order.html) can be skipped|

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you, @keharper

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed by @keharper ;)

Copy link
Contributor

Choose a reason for hiding this comment

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

@atwixfirster I can't commit the change. I can only make a suggestion to fix it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure how but you fixed that bug. I've pulled the changes and detected what md extension has been replaced with html.

Please run tests now.

Thank you!

@keharper
Copy link
Contributor

running tests

@keharper keharper merged commit 7fa9962 into magento:develop Aug 19, 2019
@ghost
Copy link

ghost commented Aug 19, 2019

Hi @atwixfirster, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

@keharper
Copy link
Contributor

Thanks @atwixfirster

@meker12
Copy link
Contributor

meker12 commented Aug 19, 2019

@keharper Please add the whatsnew for this PR. Thx.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Community Docs impacted by community code contribution Major Update Significant original updates to existing content Partner: Atwix partners-contribution PR created by Magento partner xx2.3.3 Magento 2.3.3 changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants