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

Skip to content

add validation on complete order#12658

Merged
GSadee merged 2 commits into
Sylius:masterfrom
AdamKasp:validation-placing-order-with-item-out-of-the-stock
May 24, 2021
Merged

add validation on complete order#12658
GSadee merged 2 commits into
Sylius:masterfrom
AdamKasp:validation-placing-order-with-item-out-of-the-stock

Conversation

@AdamKasp
Copy link
Copy Markdown
Contributor

Q A
Branch? master
New feature? yes
License MIT

@AdamKasp AdamKasp added the API APIs related issues and PRs. label May 20, 2021
@AdamKasp AdamKasp requested a review from a team as a code owner May 20, 2021 13:14
Comment thread src/Sylius/Behat/Context/Api/Shop/CheckoutContext.php Outdated
Comment thread src/Sylius/Behat/Resources/config/suites/api/inventory/checkout_inventory.yaml Outdated
Comment thread src/Sylius/Behat/Resources/config/suites/api/inventory/checkout_inventory.yaml Outdated
@AdamKasp AdamKasp force-pushed the validation-placing-order-with-item-out-of-the-stock branch from b2ed565 to de26201 Compare May 21, 2021 05:54
@AdamKasp AdamKasp force-pushed the validation-placing-order-with-item-out-of-the-stock branch from de26201 to e476f39 Compare May 21, 2021 07:37
@AdamKasp AdamKasp force-pushed the validation-placing-order-with-item-out-of-the-stock branch from e476f39 to 2e27e12 Compare May 21, 2021 08:45
Scenario: Placing an order with products that have sufficient quantity
Given I have added 3 products "Iron Maiden T-Shirt" to the cart
And I have proceeded selecting "Offline" payment method
And I proceed through checkout process
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
And I proceed through checkout process
When I proceed through checkout process

but why did you change that step?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I had a problem with addresses, in behats they were empty, and I get SM error, with this step, the scenario is still valid and checkout works as I expect.

Comment thread src/Sylius/Behat/Context/Api/Shop/CheckoutContext.php Outdated
@AdamKasp AdamKasp force-pushed the validation-placing-order-with-item-out-of-the-stock branch 3 times, most recently from 8e79960 to 6725511 Compare May 23, 2021 15:57
@AdamKasp AdamKasp force-pushed the validation-placing-order-with-item-out-of-the-stock branch from 6725511 to d5302d6 Compare May 23, 2021 17:29
@GSadee GSadee merged commit 9963946 into Sylius:master May 24, 2021
@GSadee
Copy link
Copy Markdown
Member

GSadee commented May 24, 2021

Thanks, Adam! 🎉

@AdamKasp AdamKasp deleted the validation-placing-order-with-item-out-of-the-stock branch July 19, 2021 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API APIs related issues and PRs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants