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

[CatalogPromotions] Showing catalog promotions in Products index and show page #13101

Merged
merged 8 commits into from Sep 15, 2021

Conversation

SirDomin
Copy link
Contributor

@SirDomin SirDomin commented Sep 14, 2021

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

Products index:

Zrzut ekranu 2021-09-15 o 14 17 50

Product's details:

Zrzut ekranu 2021-09-15 o 14 17 38

@SirDomin SirDomin requested a review from a team as a code owner September 14, 2021 12:24
@probot-autolabeler probot-autolabeler bot added the Shop ShopBundle related issues and PRs. label Sep 14, 2021
@Zales0123 Zales0123 changed the title [CatalogPromotions] add ui scenarios [WIP][CatalogPromotions] Showing catalog promotions in Products index and show page Sep 15, 2021
@Zales0123 Zales0123 changed the title [WIP][CatalogPromotions] Showing catalog promotions in Products index and show page [CatalogPromotions] Showing catalog promotions in Products index and show page Sep 15, 2021
@Zales0123 Zales0123 added the UX Issues and PRs aimed at improving User eXperience. label Sep 15, 2021
@AdamKasp AdamKasp merged commit 243d975 into Sylius:master Sep 15, 2021
@AdamKasp
Copy link
Contributor

AdamKasp commented Sep 15, 2021

Thank you, @SirDomin and @Zales0123 ! 🎉

GSadee added a commit that referenced this pull request Sep 16, 2021
…iption in label if not set (Zales0123)

This PR was merged into the 1.11-dev branch.

Discussion
----------

| Q               | A
| --------------- | -----
| Branch?         | master
| Bug fix?        | yes
| New feature?    | no
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | introduced in #13101
| License         | MIT

Before:

<img width="588" alt="Zrzut ekranu 2021-09-16 o 08 59 13" src="https://user-images.githubusercontent.com/6212718/133565044-a6ff33f4-2236-4e50-b738-fc7c27e4f774.png">


After:

<img width="607" alt="Zrzut ekranu 2021-09-16 o 08 57 03" src="https://user-images.githubusercontent.com/6212718/133565003-6b0c7c39-eafb-4724-8894-7b9daed531e6.png">


Commits
-------

4f23bb2 Do not display applied promotion description in label if not set
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Shop ShopBundle related issues and PRs. UX Issues and PRs aimed at improving User eXperience.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants