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

Filters are not case insensitive #107

Open
bakkerthehacker opened this issue Oct 14, 2022 · 2 comments
Open

Filters are not case insensitive #107

bakkerthehacker opened this issue Oct 14, 2022 · 2 comments

Comments

@bakkerthehacker
Copy link
Contributor

In previous versions of admin sequelizejs, filtering could be done using upper or lower case letters. This functionality no longer works in later versions of admin sequelizejs.

Instead of being able to find similar text that can ignore case, the exact case must be known when searching. This is problematic as searches that previously worked no longer work correctly, and additional work must be done to find the exact case of text when searching.

While this used to work in older versions, I also see that it was removed to be compatible with mysql. Compatibility is useful, but I don't feel like the existing features should be given up in this case.

Versions 1.2.1 and versions 2.0.0 are working correctly and do not have the problematic filtering. I would like to continue updating admin packages to receive new features and fixes, but for now will be stuck on these versions.

@fabiosantoscode
Copy link

Hey there! I feel partially responsible, as I posted an issue that was fixed by removing case insensitive searches.

#65

giwiro added a commit to giwiro/adminjs-sequelizejs that referenced this issue Mar 10, 2023
@giwiro
Copy link

giwiro commented Mar 10, 2023

How about a check for postgresql ? I sent a PR #118

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