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

Using Form request validation for Nova resource #106

Closed
peanut24 opened this issue Aug 24, 2018 · 6 comments
Closed

Using Form request validation for Nova resource #106

peanut24 opened this issue Aug 24, 2018 · 6 comments

Comments

@peanut24
Copy link

@peanut24 peanut24 commented Aug 24, 2018

I have existing form request validation classes that I'm using for storing a user for instance, is it possible to reuse these in my Nova resource, or do you have to specify the validation rules per field? I can't find reference to this in the documentation for Nova.

Thanks

@stevelacey
Copy link

@stevelacey stevelacey commented Sep 4, 2018

Also would make it possible to validate arrays, which I am not sure is possible via the validation rules per field implementation

@davidhemphill
Copy link
Contributor

@davidhemphill davidhemphill commented Oct 26, 2018

This is not currently supported.

@fbingha
Copy link

@fbingha fbingha commented Nov 7, 2018

Is this planned?

@yehudahkay
Copy link

@yehudahkay yehudahkay commented Mar 6, 2019

You can add a rules array on either the model or form request and then reference it in the nova controller, it would be nice if this would work automatically though.

@EricPaulson
Copy link

@EricPaulson EricPaulson commented Apr 30, 2019

It doesn't make sense for Nova to require its own homegrown validation rather than using Laravel's perfectly functional Requests for validation. This requires many of us to duplicate logic in our applications. When will this be addressed?

@davidhemphill
Copy link
Contributor

@davidhemphill davidhemphill commented May 1, 2019

Don't know why folks don't like that I said it's currently not supported. It's the truth. 😄

Would welcome a PR to add this, otherwise there are more pressing features/bugs to address at this time.

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

Successfully merging a pull request may close this issue.

None yet
6 participants