Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Simplify inventory REST tutorial #7583

Merged
merged 12 commits into from
Jul 24, 2020
Merged

Simplify inventory REST tutorial #7583

merged 12 commits into from
Jul 24, 2020

Conversation

keharper
Copy link
Contributor

@keharper keharper commented Jul 23, 2020

Purpose of this pull request

#7545 updated the Inventory REST tutorial to be compatible with the new in-store pickup feature, but it was causing some problems, and I couldn't complete the GraphQL ISP tutorial. This PR simplifies the REST tutorial by performing all steps on the default store view. As a side effect of this, I replaced all references to German warehouses/stores with entities in California

I added a substep to explain where to get order item IDs. I also deleted a substep that featured the /V1/shipment endpoint, which is obsolete.

Affected DevDocs pages

https://devdocs.magento.com/guides/v2.4/rest/tutorials/inventory/index.html

Links to Magento source code

  • ...

@keharper keharper requested a review from jeff-matthews July 24, 2020 03:51
@keharper
Copy link
Contributor Author

running tests

@keharper
Copy link
Contributor Author

running tests

@keharper
Copy link
Contributor Author

running tests

@keharper keharper merged commit 1465000 into master Jul 24, 2020
@ghost
Copy link

ghost commented Jul 24, 2020

Hi @keharper, 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 keharper deleted the kh_simplify-inv-rest branch July 26, 2020 18:01
@jeff-matthews jeff-matthews added the Internal Dev Differentiates work between community and Magento staff label Aug 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Internal Dev Differentiates work between community and Magento staff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants