-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Improve Order get endpoints #16321
Improve Order get endpoints #16321
Conversation
TheMilek
commented
May 29, 2024
Q | A |
---|---|
Branch? | 1.12 |
Bug fix? | yes |
New feature? | no |
BC breaks? | no |
License | MIT |
Bunnyshell Preview Environment deletedAvailable commands:
|
801d39a
to
bb3cc9d
Compare
ca27091
to
3d42fd4
Compare
} | ||
|
||
/** @test */ | ||
public function it_gets_order_item_adjustments(): void |
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.
Since we already have a split in the actions between the user/visitor, maybe we should also duplicate all the other tests in this file per case.
src/Sylius/Bundle/ApiBundle/DataProvider/OrderAdjustmentsSubresourceDataProvider.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Resources/config/services/security.xml
Outdated
Show resolved
Hide resolved
...romotion/receiving_discount/receiving_discounts_with_product_minimum_price_specified.feature
Show resolved
Hide resolved
3d42fd4
to
fe653b7
Compare
fe653b7
to
5bba8f0
Compare
xsi:schemaLocation="http://symfony.com/schema/dic/services http://symfony.com/schema/dic/services/services-1.0.xsd" | ||
> | ||
<services> | ||
<service id="sylius_api.security.voter.order" class="Sylius\Bundle\ApiBundle\Security\OrderAdjustmentsVoter"> |
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.
<service id="sylius_api.security.voter.order" class="Sylius\Bundle\ApiBundle\Security\OrderAdjustmentsVoter"> | |
<service id="sylius_api.security.voter.order_adjustments" class="Sylius\Bundle\ApiBundle\Security\OrderAdjustmentsVoter"> |
if ($this->adjustmentOrderProvider->provide($subjectItem)) { | ||
/** @var OrderInterface $order */ | ||
$order = $this->adjustmentOrderProvider->provide($subjectItem); |
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.
if ($this->adjustmentOrderProvider->provide($subjectItem)) { | |
/** @var OrderInterface $order */ | |
$order = $this->adjustmentOrderProvider->provide($subjectItem); | |
$order = $this->adjustmentOrderProvider->provide($subjectItem); | |
if ($order !== null) { |
Thank you, Kamil! 🥇 |
…Sadee) This PR was merged into the 1.13 branch. Discussion ---------- | Q | A |-----------------|----- | Branch? | 1.13 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no<!-- don't forget to update the UPGRADE-*.md file --> | Related tickets | after upmerge of #16321 | License | MIT <!-- - Bug fixes must be submitted against the 1.12 or 1.13 branches - Features and deprecations must be submitted against the 1.14 branch - Features, removing deprecations and BC breaks must be submitted against the 2.0 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 ------- [API] Adjust adjustments contract tests after upmerge