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
[API] Refactor converting arguments of commands from IRIs to identifiers #12769
[API] Refactor converting arguments of commands from IRIs to identifiers #12769
Conversation
Tomanhez
commented
Jul 1, 2021
•
edited by GSadee
edited by GSadee
Q | A |
---|---|
Branch? | master |
Bug fix? | no |
New feature? | yes |
BC breaks? | no |
Deprecations? | no |
Related tickets | #12487 |
License | MIT |
0adf12d
to
a81f6c0
Compare
src/Sylius/Bundle/ApiBundle/spec/Serializer/CommandFieldItemIriToIdentifierDenormalizerSpec.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Serializer/CommandFieldItemIriToIdentifierDenormalizer.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Serializer/CommandFieldItemIriToIdentifierDenormalizer.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Serializer/CommandFieldItemIriToIdentifierDenormalizer.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Serializer/CommandFieldItemIriToIdentifierDenormalizer.php
Outdated
Show resolved
Hide resolved
4da7547
to
b91662f
Compare
src/Sylius/Bundle/ApiBundle/Serializer/CommandFieldItemIriToIdentifierDenormalizer.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Serializer/CommandFieldItemIriToIdentifierDenormalizer.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Serializer/CommandFieldItemIriToIdentifierDenormalizer.php
Outdated
Show resolved
Hide resolved
3d1d0b5
to
29df650
Compare
c98f6a1
to
3cc47a7
Compare
3cc47a7
to
5f45010
Compare
92eabae
to
7aa9b4d
Compare
0c504cd
to
a87ef15
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.
I'm wondering if we shouldn't
use Sylius\Bundle\ApiBundle\Command\OrderTokenValueAwareInterface; | ||
use Sylius\Bundle\ApiBundle\Command\SubresourceIdAwareInterface; | ||
|
||
/** @experimental */ | ||
class ChooseShippingMethod implements OrderTokenValueAwareInterface, SubresourceIdAwareInterface | ||
class ChooseShippingMethod implements OrderTokenValueAwareInterface, SubresourceIdAwareInterface, IriToIdentifierConversionAwareInterface |
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 wondering if both SubresourceIdAwareInterface and IriToIdentifierConversionAwareInterface inferfaces shouldn't be extracted to one. Same for most of usage of IriToIdentifierConversionAwareInterface
Thanks, Tomasz and Grzegorz! 🥇 |