-
Notifications
You must be signed in to change notification settings - Fork 61
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
feat: add fuzzy param as optional #2463
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
You can test in this env https://sfj-f6292be--jwpepperdev.preview.vtex.app/ |
Hello everyone! I mentioned this in the thread, but I wanted to share it here for better visibility: Thank you for suggesting this fix ✨ . However, this approach is not aligned with the recommendations from the IS team. Here’s the expected behavior:
We might need to make further adjustments to fix this issue, as we are currently using it incorrectly 🙇🏻♀️ |
What's the purpose of this pull request?
adds the fuzzy parameter as an option so that the search result is more precise if the store needs it
How it works?
Basically the search api already accepts this parameter in the api, I added the same parameter to the faststore queries so stores can pass the route if they want
How to test it?
Access a search page or category by passing the parameter fuzzy=0 or fuzzy=1 and check if the result has changed