
~A couple of things to do here.~
~### API~
~1. Make sure the unique can only be set for text fields. BTW, because of this, I think the unique should be a separate Commodo field (packages/api-headless-cms/src/handler/plugins/models/ContentModel/createFieldsModel.ts:23). Since it's a field-specific option, I would put it into options field.~
~1.1 Because of the change above, there will probably be some adjustments in the GraphQL schema creation process.~
~2. Make sure that if a user makes a field unique, that he cannot submit another entry with the same value for that field.~
~###Â App
~1. Add description below in the dialog, explaining what unique actually means.~
Create a "unique" field validator.
@doitadrian Im wondering why you decided to specify this as it's own field on the form-field itself. I'd argue that this would really act better and be more extendable as an additional validator option
You are right, this will be a separate validator. :)
On Mon, 8 Jun 2020 at 15:34, Jon R notifications@github.com wrote:
@doitadrian https://github.com/doitadrian Im wondering why you decided
to specify this as it's own field on the form-field itself. I'd argue that
this would really act better and be more extendable as an additional
validator option—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/webiny/webiny-js/issues/822#issuecomment-640606075,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHCI7CUXO36PNRRMS4PMB3RVTSFBANCNFSM4M2RDFFQ
.
--
Adrian S.
Please reopen this issue once you add more information and updates here. If this is not the case and you need some help, feel free to seek help from our Slack Community or ping one of the reviewers. Thank you for your contributions!
This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days.
Most helpful comment
You are right, this will be a separate validator. :)
On Mon, 8 Jun 2020 at 15:34, Jon R notifications@github.com wrote:
--
Adrian S.