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

Skip to content

[Catalog Promotion] Cart catalog promotion becomes ineligible#13112

Merged
SirDomin merged 2 commits into
Sylius:masterfrom
AdamKasp:cart-catalog-promotion-becomes-ineligible
Sep 15, 2021
Merged

[Catalog Promotion] Cart catalog promotion becomes ineligible#13112
SirDomin merged 2 commits into
Sylius:masterfrom
AdamKasp:cart-catalog-promotion-becomes-ineligible

Conversation

@AdamKasp
Copy link
Copy Markdown
Contributor

Q A
Branch? master
Bug fix? no
New feature? no
BC breaks? no
License MIT

@AdamKasp AdamKasp added the API APIs related issues and PRs. label Sep 15, 2021
@AdamKasp AdamKasp requested a review from a team as a code owner September 15, 2021 12:59
Feature: Seeing current prices of products after catalog promotion becomes ineligible
In order to buy products in its correct prices
As a Customer
I want to have products with its current prices in the cart
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I want to have products with their original prices in the cart?

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.

the current sentence has more sense for me because in this scenarios I want to have a valid and current price, for customer, it is irrelevant if it is original or discounted (by different catalog promotion for example)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I was referring that the price that Customer sees is always current for they :) but it's not relevant for know, I would only change its to their 🖖

And I specified the billing address as "Ankh Morpork", "Frost Alley", "90210", "United States" for "Jon Snow"
And I proceed through checkout process
Then I should be on the checkout summary step
And I should see product "T-Shirt" with unit price "$20.00"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cannot we check it before the final stap? Even on the cart itself?

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.

TBH yes we can :D, we need to run order Processor after admin changed something so each step after addressing is valid, going through the entire checkout was easy in behat and also it checks what I want :)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe we could just update the cart? It's not a blocker for merge, but with all these steps it seems like it's necessary to go through the checkout to see the result. And there is in the cart in the scenario's description 💃

Comment thread src/Sylius/Behat/Context/Api/Shop/CheckoutContext.php Outdated
@AdamKasp AdamKasp force-pushed the cart-catalog-promotion-becomes-ineligible branch from 6e707be to a285b53 Compare September 15, 2021 13:39
@AdamKasp AdamKasp force-pushed the cart-catalog-promotion-becomes-ineligible branch from a285b53 to fd4886e Compare September 15, 2021 13:41
@SirDomin SirDomin merged commit d43e908 into Sylius:master Sep 15, 2021
@SirDomin
Copy link
Copy Markdown
Contributor

Thank you, Adam! 🎉

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