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
[BUG] The hostname resolver does not check for the channel status #14155
[BUG] The hostname resolver does not check for the channel status #14155
Conversation
src/Sylius/Component/Channel/Repository/ChannelRepositoryInterface.php
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @jakubtobiasz!
@lchrusciel I've changed a little the scenario to add API easier. I hope it still sounds good :). |
src/Sylius/Behat/Resources/config/suites/api/channel/channels.yml
Outdated
Show resolved
Hide resolved
*/ | ||
public function iShouldNotSeeProduct(string $productName): void | ||
{ | ||
Assert::true(!in_array($productName, $this->homePage->getLatestProductsNames())); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm genuinely surprised there is no Assert::notInArray
😱
Thank you, Jakub! 🎉 |
This PR was merged into the 1.11 branch. Discussion ---------- | Q | A | |-----------------|--------------------------------------------------------------| | Branch? | 1.11| | Bug fix? | yes | | New feature? | no | | BC breaks? | no | | Deprecations? | no| | Related tickets | after #14155 | License | MIT | <!-- - Bug fixes must be submitted against the 1.11 or 1.12 branch - Features and deprecations must be submitted against the 1.13 branch - Make sure that the correct base branch is set To be sure you are not breaking any Backward Compatibilities, check the documentation: https://docs.sylius.com/en/latest/book/organization/backward-compatibility-promise.html --> Commits ------- 19db727 [Behat][API] Add missing channels suite to imports
What has been done:
UPGRADE-1.11.md
and theUPGRADE-1.10.md
filesWe've had a discussion about parametrizing the
HostnameBasedRequestResolver
(what I've done btw 😂) but we decided to drop that idea and keep the original concept.Thank you @lruozzi9 for your PR we could base on :).