forked from Arteha/admin-bro-typeorm
-
Notifications
You must be signed in to change notification settings - Fork 35
Open
Description
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.
adminjs-typeorm/src/Property.ts
Line 17 in b1c6076
| public isEditable(): boolean { |
"they" are doing it with sequelize too:
https://github.com/SoftwareBrothers/adminjs-sequelizejs/pull/64/files
Thanks
Metadata
Metadata
Assignees
Labels
No labels