Skip to content

Inconsistent Product Sorting by Price on Shop Page for Vendor-Added Products #2431

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

Open
sazz-pranto opened this issue Nov 10, 2024 · 4 comments
Assignees

Comments

@sazz-pranto
Copy link

sazz-pranto commented Nov 10, 2024

Bug Description

When products are added from the Admin Panel, they can be sorted by price (from high to low and vice versa) correctly on the Shop Page. However, products added from the Vendor Dashboard do not sort correctly by price on the Shop Page. For a better understanding, please have a look at this video: https://d.pr/v/RGgZzy

Step To Reproduce

  1. From the Vendor Dashboard, add several products with varying prices.
  2. Go to the Shop Page and attempt to sort the products by price (high to low or low to high).
  3. Observe the sorting behavior.

Expected Behaviour

Products added from the Vendor Dashboard should be sorted by price accurately on the Shop page, just as they are for products added through the Admin Panel.

Actual Behaviour

As shown in the video.

Additional Information

N/A

Environment (please complete the following information)

Dokan: v3.13.0
WooCommerce: v9.3.3
WordPress: v6.6.2
@mrabbani
Copy link
Member

@sazz-pranto vai, It is addressed in #2098

@TanvirHasan19
Copy link
Member

@jamil-mahmud bhai || @mrabbani bhai || @kzamanbd bhai,

The issue still persists in the latest version of the Dokan Lite plugin. I kindly request that you investigate this matter and implement a proper solution as soon as possible to address the incompatibility between Dokan and WooCommerce's default product sorting options.

We've already received a couple of reports from clients experiencing this issue. Your prompt attention to this matter would be greatly appreciated.

Thanks in advance!

@TanvirHasan19
Copy link
Member

@jamil-mahmud Bhai || @kzamanbd Bhai,

I've created a snippet that temporarily resolves the issue. You can find the snippet at the following link: https://gist.github.com/TanvirHasan19/91dd4362d19072337ca9b02796e5ef07

@kzamanbd
Copy link
Contributor

@TanvirHasan19 bhai, Thanks for implementing the temporary solution — really appreciate your quick action!

We have already implemented a more permanent fix in this PR: #2098. A big thanks to @devAsadNur bhai for creating this PR, and also to you for handling the temporary solution so efficiently. There are a few technical dependencies we are currently working through, and once resolved, we’ll release it soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants