SoftwareBrothers / adminjs-sequelizejs

Sequelizejs adapter for AdminBro
MIT License
27 stars 34 forks source link

Filters are not case insensitive #107

Open bakkerthehacker opened 1 year ago

bakkerthehacker commented 1 year ago

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 commented 1 year ago

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

https://github.com/SoftwareBrothers/adminjs-sequelizejs/issues/65

giwiro commented 1 year ago

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