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
[Migration Carrier pages] CQRS for Add / Get / Update / Upload logo #36063
[Migration Carrier pages] CQRS for Add / Get / Update / Upload logo #36063
Conversation
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 @boherm
I added a lot of comments on the PR but many of them can be postponed in a later PRs, the current PR is already a very good start that could be merged early so we can start parellizing the job We can discuss what should and should not be fixed right now 😉
src/Adapter/Carrier/CommandHandler/DeleteCarrierLogoHandler.php
Outdated
Show resolved
Hide resolved
src/Adapter/Carrier/CommandHandler/DeleteCarrierLogoHandler.php
Outdated
Show resolved
Hide resolved
src/Adapter/Carrier/CommandHandler/UploadCarrierLogoHandler.php
Outdated
Show resolved
Hide resolved
src/Adapter/Carrier/QueryHandler/GetCarrierForEditingHandler.php
Outdated
Show resolved
Hide resolved
src/PrestaShopBundle/Resources/config/services/adapter/carrier.yml
Outdated
Show resolved
Hide resolved
tests/Integration/Behaviour/Features/Context/Domain/CarrierFeatureContext.php
Show resolved
Hide resolved
tests/Integration/Behaviour/Features/Context/Domain/CarrierFeatureContext.php
Outdated
Show resolved
Hide resolved
tests/Integration/Behaviour/Features/Context/Domain/CarrierFeatureContext.php
Show resolved
Hide resolved
tests/Integration/Behaviour/Features/Scenario/Carrier/carrier_management.feature
Show resolved
Hide resolved
8cc3709
to
d11abc6
Compare
d11abc6
to
50b75e4
Compare
tests/Integration/Behaviour/Features/Scenario/Carrier/carrier_management.feature
Show resolved
Hide resolved
50b75e4
to
fea88f1
Compare
fea88f1
to
45f7fb2
Compare
45f7fb2
to
1bafd6d
Compare
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 @boherm
This can't be tested by the QA team, the behat tests are enough for now. |
Fix #35925