-
Notifications
You must be signed in to change notification settings - Fork 103
SUPESC-990 Adjust documentation for review. #3358
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
SUPESC-990 Adjust documentation for review. #3358
Conversation
✅ Deploy Preview for spryker-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
matweew
left a comment
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.
we don't need to create a copy of integration guides in 202507.0 folder anymore (this version is already locked to the product release version), we modify only latest
so please remove new pages
.../all/install-features/latest/install-the-product-group-product-rating-and-reviews-feature.md
Outdated
Show resolved
Hide resolved
| Product reviews are displayed on the catalog page, search page, and product detail page. To optimize performance when displaying product ratings, use the `ProductReviewStorageProductViewExpanderPlugin` instead of `ProductReviewSummaryProductViewBulkExpanderPlugin`. | ||
|
|
||
| The new plugin retrieves product ratings from storage in bulk, which is more efficient than fetching reviews from a search source and calculating ratings based on reviews. |
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.
Please, suggest which package versions we need to update to have this
| Update `spryker-shop/product-review-widget` to version `^1.17.0` or higher: | ||
|
|
||
| ```bash | ||
| composer require spryker-shop/product-review-widget:"^1.17.0" --update-with-dependencies |
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.
please, never suggest --update-with-dependencies
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.
PR Description
Describe the context for your changes and the changes you've made.
Tickets
https://spryker.atlassian.net/browse/SUPESC-990
Checklist