SoftwareBrothers / adminjs-sequelizejs

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

[Fix] Return correct field name when using underscored: true #23

Closed ksewo closed 4 years ago

ksewo commented 4 years ago

When using underscored: true, sequelizeField.field returns field name from database (i.e. first_name), but sequelizeField.fieldName returns correct name for underscored option in model (firstName), so every field name in model that differs from database name will not get returned when using SELECT and it is impossible to update or delete such a field.

Related PR: https://github.com/SoftwareBrothers/admin-bro-sequelizejs/pull/24

ksewo commented 4 years ago

@wojtek-krysiak

ksewo commented 4 years ago

Any updates or is this library inactive?

wojtek-krysiak commented 4 years ago

merged and published. Sorry for the delay. I simply missed that repo. You can jump on our slack channel https://join.slack.com/t/adminbro/shared_invite/zt-djsqxxpz-_YCS8UMtQ9Ade6DPuLR7Zw and ping me if you have any questions/urgent requests