Skip to content
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

In Customer Profile Reviews -> there we are able to see only 5 reviews in total, but I have approved more reviews for the same user. #9568

Closed
1 task done
ashishkumar-webkul opened this issue Mar 5, 2024 · 1 comment
Assignees
Labels
Bug Fixed When bug issue is fixed. Bug Something isn't working.

Comments

@ashishkumar-webkul
Copy link
Collaborator

Terms

  • Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected

2.1.1

Issue Description

When we check the customer profile review section, there we can see only 5 reviews are appearing. But for the same user I have created more then 10 approved reviews, I am unable to see them.

Preconditions

  1. User must be logged in.
  2. Reviews to different products must be done.
  3. Minimum 10 reviews for the same user.

Steps to reproduce

  1. Go to Admin ->Customers -> reviews -> Approve.
  2. Go to User profile -> Reviews -> Check the reviews details.
  3. There we can see only 5 reviews. Rest reviews can be seen in product page.

Expected Result

All the reviews which are approved must be seen in the customer profile as well.

Actual Result

Currently we can see that only 5 reviews are visible. Please find the video for reference.

Video

Reviews.mp4

Screenshots

No response

Additional context

No response

@ashishkumar-webkul ashishkumar-webkul added the Bug Something isn't working. label Mar 5, 2024
@shivendra-webkul shivendra-webkul self-assigned this Mar 5, 2024
shivendra-webkul added a commit to shivendra-webkul/bagisto that referenced this issue Mar 6, 2024
shivendra-webkul added a commit to shivendra-webkul/bagisto that referenced this issue Mar 6, 2024
@shivendra-webkul shivendra-webkul added the Fix Proposed A issue that has a PR related to it that provides a possible resolution but stil needs to be merged. label Mar 6, 2024
@ashishkumar-webkul
Copy link
Collaborator Author

Checked and Found that the issue has been fixed. Pagination has been implemented in the same. It is working fine in all locale.

Please fine the video below.

Video

Otherlocale.mp4
Reviews.added.mp4

@ashishkumar-webkul ashishkumar-webkul added the QA Passed When QA pass the pull request. label Mar 6, 2024
@jitendra-webkul jitendra-webkul added the Bug Fixed When bug issue is fixed. label Mar 7, 2024
@devansh-webkul devansh-webkul removed QA Passed When QA pass the pull request. Fix Proposed A issue that has a PR related to it that provides a possible resolution but stil needs to be merged. labels May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Fixed When bug issue is fixed. Bug Something isn't working.
Projects
None yet
Development

No branches or pull requests

4 participants