Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make LocalFile field required? #491

Closed
alsoicode opened this issue Jul 25, 2014 · 2 comments
Closed

Make LocalFile field required? #491

alsoicode opened this issue Jul 25, 2014 · 2 comments
Assignees

Comments

@alsoicode
Copy link
Contributor

Is there a way to set a LocalFile or other file-field type(s) to required = true?

I've tried just adding the property to the schema, but it's not affecting form validation.

@grabbou
Copy link
Contributor

grabbou commented Aug 11, 2014

Haven't checked it yet, but it's coming in #526 by default (we will make sure it works).
FYI -> With release of new type Field, LocalFile will be depreciated. New class will give you lots of new options to set that you might enjoy.

I will give it a go later today and get back to you. If it's an easy one - maybe I will roll out PR for that.

@creynders
Copy link
Contributor

We're closing all feature requests to keep the issue tracker unpolluted. I've moved this to ProductPains:https://productpains.com/post/keystonejs/allow-making-localfile-field-required

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants