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.

Sign up for GitHub

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

Jump to bottom

Only the review should be deleted which the user wants to delete. #6459

Closed
Sanjaybhattwebkul opened this issue May 31, 2022 · 2 comments
Closed

Only the review should be deleted which the user wants to delete. #6459

Sanjaybhattwebkul opened this issue May 31, 2022 · 2 comments
Assignees
@Sanjaybhattwebkul
Labels
Bug Fixed When bug issue is fixed. Bug Something isn't working. Shop Shop package related issues.
Milestone
1.4.4

Comments

@Sanjaybhattwebkul
Copy link
Contributor

Bagisto Version: Master

Steps To reproduce--

1. Log in as a customer.
2. Click on the reviews inside the profile section.
3. Try to delete any review.

Expected result: Only the review should be deleted which the user wants to delete.

Actual result: The review is deleted in ascending order if the user is trying to delete the review. Please find the screen recording Here
@Sanjaybhattwebkul Sanjaybhattwebkul added Bug Something isn't working. Shop Shop package related issues. labels May 31, 2022
@Sanjaybhattwebkul Sanjaybhattwebkul self-assigned this May 31, 2022
@Sanjaybhattwebkul Sanjaybhattwebkul added this to the 1.4.4 milestone May 31, 2022
Sanjaybhattwebkul pushed a commit to Sanjaybhattwebkul/bagisto that referenced this issue May 31, 2022
Issue bagisto#6459 fixed
6290a16
@Sanjaybhattwebkul Sanjaybhattwebkul mentioned this issue May 31, 2022
Merged
@Sanjaybhattwebkul Sanjaybhattwebkul added Fix Proposed A issue that has a PR related to it that provides a possible resolution but stil needs to be merged. and removed In Progress labels May 31, 2022
jitendra-webkul added a commit that referenced this issue Jun 8, 2022
@jitendra-webkul
Merge pull request #6460 from Sanjaybhattwebkul/delete-review-issue
dc834b0 
Issue #6459 fixed
@Sanjaybhattwebkul
Copy link
Contributor Author

I am getting the same issue in the velocity theme

@Sanjaybhattwebkul Sanjaybhattwebkul added In Progress and removed Fix Proposed A issue that has a PR related to it that provides a possible resolution but stil needs to be merged. labels Jun 8, 2022
Sanjaybhattwebkul pushed a commit to Sanjaybhattwebkul/bagisto that referenced this issue Jun 9, 2022
Issue bagisto#6459-fixed-default-theme
add4be2
@Sanjaybhattwebkul Sanjaybhattwebkul mentioned this issue Jun 9, 2022
Merged
@Sanjaybhattwebkul Sanjaybhattwebkul added Fix Proposed A issue that has a PR related to it that provides a possible resolution but stil needs to be merged. and removed In Progress labels Jun 9, 2022
jitendra-webkul added a commit that referenced this issue Jun 9, 2022
@jitendra-webkul
Merge pull request #6478 from Sanjaybhattwebkul/delete-review-issue
f4d81ec 
Issue #6459-fixed-default-theme
@Sanjaybhattwebkul Sanjaybhattwebkul added Bug Fixed When bug issue is fixed. and removed Fix Proposed A issue that has a PR related to it that provides a possible resolution but stil needs to be merged. labels Jun 10, 2022
@Sanjaybhattwebkul
Copy link
Contributor Author

@jitendra-webkul
Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here, for future reference.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Sanjaybhattwebkul Sanjaybhattwebkul

Labels
Bug Fixed When bug issue is fixed. Bug Something isn't working. Shop Shop package related issues.
Projects
None yet
Milestone
1.4.4
Development

No branches or pull requests
1 participant
@Sanjaybhattwebkul