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

[BUG] Filter product #551

Open
leandrosantoss opened this issue May 27, 2024 · 10 comments
Open

[BUG] Filter product #551

leandrosantoss opened this issue May 27, 2024 · 10 comments
Assignees

Comments

@leandrosantoss
Copy link

Describe the bug
Click on product registration and try to filter

I have 3 products
image

But when filter only accepts the full name. If filtering a snippet doesn't work
image

image

Not case insensitive
image

Thanks

@hussaincoder23
Copy link

Can you please describe it again , what is the issue ?

@leandrosantoss
Copy link
Author

The problem is in the filter:

  • Case sensitive
  • Filters only full name

Should be:

  • Case insensitive
  • Filter by a part of the name (show prints attached)

@treoden
Copy link
Collaborator

treoden commented May 29, 2024

Hi @leandrosantoss ,

You are using keyword search, it is a full text search in postgres db. Not an ilike query.
You can check more here:

`to_tsvector('simple', product_description.name || ' ' || product_description.description) @@ websearch_to_tsquery('simple', :${bindingKey})`,

Thanks

@leandrosantoss
Copy link
Author

Is the idea to continue with the full text?

It makes it a little difficult because rarely does the person know the exact name.

@treoden
Copy link
Collaborator

treoden commented May 31, 2024

Hi @leandrosantoss

I tried and see it:

  1. Is not case sensitive
  2. Does not require full name

image

Can you check again?
Thanks

@leandrosantoss
Copy link
Author

Hi, @treoden

It doesn't work in my application (1.1.0). Do I need to do any configuration?

products-google-chrome-2024-06-03-11-50-24_MoZYObZd.mp4

@leandrosantoss
Copy link
Author

Hi @treoden, do you understand my problem?

@treoden
Copy link
Collaborator

treoden commented Jun 7, 2024

Hi @leandrosantoss

Yes i see. Let me double check again

@leandrosantoss
Copy link
Author

Hi @treoden
Were you able to check?

@leandrosantoss
Copy link
Author

Hi @treoden news about this?

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

No branches or pull requests

3 participants