search

SoftwareBrothers / adminjs-sequelizejs

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

Only one string field filter works #60

Closed ElvisYang closed 3 years ago

ElvisYang commented 3 years ago

When I specify 2 string fields to do the filter. it looks the result just for one field filter.

in this line https://github.com/SoftwareBrothers/admin-bro-sequelizejs/blob/master/src/utils/convert-filter.ts#L29, the existing [Op.and] of memo will always override the new value set here https://github.com/SoftwareBrothers/admin-bro-sequelizejs/blob/master/src/utils/convert-filter.ts#L23

ElvisYang commented 3 years ago

Just create a PR to fix this https://github.com/SoftwareBrothers/admin-bro-sequelizejs/pull/61

ElvisYang commented 3 years ago

This issue is fixed by https://github.com/SoftwareBrothers/admin-bro-sequelizejs/pull/61