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

Layered navigation filters not working #7273

Closed
1 task done
tahiryasin opened this issue Feb 7, 2023 · 6 comments
Closed
1 task done

Layered navigation filters not working #7273

tahiryasin opened this issue Feb 7, 2023 · 6 comments
Labels
Bug Fixed When bug issue is fixed. Bug Something isn't working.
Milestone

Comments

@tahiryasin
Copy link

Terms

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

Bagisto Version(s) affected

0.1.0

Issue Description

Layered navigation filters not working.

final-filters

Preconditions

Open any category page

Steps to reproduce

  1. Go to any category page from main menu
  2. Apply any filters, like price, color or brand
  3. No results will show

Expected Result

Filtered products should show

Actual Result

No products showing

Screenshots

image

Additional context

No response

@tahiryasin tahiryasin added the Bug Something isn't working. label Feb 7, 2023
@Sanjaybhattwebkul Sanjaybhattwebkul added Not An Issue Not getting this issue / This isn't an issue. Fix Proposed A issue that has a PR related to it that provides a possible resolution but stil needs to be merged. and removed Not An Issue Not getting this issue / This isn't an issue. labels Feb 8, 2023
@Sanjaybhattwebkul
Copy link
Contributor

Hi there,
This issue is fixed in our master.Please find the screenshot Here

@tahiryasin
Copy link
Author

@Sanjaybhattwebkul Appreciate your response.
Kindly tag the affected files so that we can apply the same fix in our project too.

@tahiryasin
Copy link
Author

tahiryasin commented Feb 8, 2023

@Sanjaybhattwebkul
The issue is still present, even your official demo has this issue as of today
Kindly do not close the tickets without proper testing and resolution.

https://demo.bagisto.com/bagisto-common/men?price=0,398

image

@saurav-webkul
Copy link
Member

Hi @tahiryasin

As we told the issue has been fixed in the master branch. The demo consists of the latest stable version which is v1.4.5 so you are getting the issues. But rest assured this has been fixed and in the next release that is about to come, it will be shown as fixed. Once the release will come then we will update the demo.

@tahiryasin
Copy link
Author

@saurav-webkul @Sanjaybhattwebkul

Could you please share the files that are updated

I want to apply the fix to our project too, we have purchased your Multitenant SaaS module too.

Looking forward to your support.

@Sanjaybhattwebkul
Copy link
Contributor

Hi there,
Please take a reference from here

This was referenced Feb 8, 2023
@devansh-webkul devansh-webkul 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 Apr 10, 2023
@shivendra-webkul shivendra-webkul added this to the 1.5.0 milestone Apr 21, 2023
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

5 participants