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] Product has last 3 reviews by default #12483
Conversation
SirDomin
commented
Mar 26, 2021
Q | A |
---|---|
Branch? | master |
Bug fix? | no |
New feature? | yes |
BC breaks? | no |
Deprecations? | no |
Related tickets | |
License | MIT |
8fefe1a
to
47965c3
Compare
47965c3
to
5f0ce40
Compare
src/Sylius/Bundle/ApiBundle/Resources/config/serialization/ProductReview.xml
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Resources/config/serialization/ProductReview.xml
Outdated
Show resolved
Hide resolved
@@ -20,31 +20,38 @@ | |||
<group>product_review:read</group> | |||
</attribute> | |||
<attribute name="createdAt"> | |||
<group>product_review:read</group> | |||
</attribute> | |||
<group>product_review:read</group> |
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.
As you've touched this place anyway, can we adjust previous serialization groups to admin:product_review:read
? Can be done in separate PR
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.
There is also a need for adding a filter with sorting product reviews
src/Sylius/Bundle/ApiBundle/Resources/config/serialization/ProductReview.xml
Outdated
Show resolved
Hide resolved
1fa4671
to
21b91f5
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.
And the rebase is needed
src/Sylius/Behat/Resources/config/services/contexts/api/shop.xml
Outdated
Show resolved
Hide resolved
21b91f5
to
8e74ab7
Compare
d71210c
to
849ca13
Compare
Thank you, @SirDomin! 🎉 |