-
Notifications
You must be signed in to change notification settings - Fork 80
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
"length" validation should add "required" attribute automatically #18
Comments
@kv109 Ah, yes. I agree. Are you still willing to work on a PR? |
@amatsuda Not really. I wanted you to merge my another PR first (you just did). But I have something different. On my own fork I did simple handling of |
what is the state of this? 🤔 |
I haven't installed this gem and just reading through issues before I do. Unfortunately have to disagree on this...it's a narrow/opinionated use case. If you want both |
Hi!
I've noticed that
translates to
To be 100% accurate with actual model validation, it should translate to
If you agree, I could prepare a pull request with that, are you interested?
The text was updated successfully, but these errors were encountered: