search

SoftwareBrothers / adminjs-typeorm

TypeORM adapter for AdminJS
MIT License
24 stars 31 forks source link

Primary column not editable #49

Open trangvu9 opened 1 year ago

trangvu9 commented 1 year ago

Is there a good reason for this line's logic? I think in many cases primary columns such as those non-auto-generated ones should be editable.

https://github.com/SoftwareBrothers/adminjs-typeorm/blob/b1c607624b1cf284892856bfb59012854829ed98/src/Property.ts#L17

"they" are doing it with sequelize too:

https://github.com/SoftwareBrothers/adminjs-sequelizejs/pull/64/files

Thanks

dziraf commented 1 year ago

You can make any field editable using resource options configuration