#17637: Client side validation classes for forms
-------------------------------------+-------------------------------------
Reporter: karthikabinav | Owner: Marcelo
| Galigniana
Type: New feature | Status: closed
Component: Forms | Version: 1.3
Severity: Normal | Resolution: wontfix
Keywords: Forms , Client Side | Triage Stage: Accepted
validation, js |
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Carlton Gibson):
* status: assigned => closed
* resolution: => wontfix
Comment:
Hi Marcelo. Thanks for the input on this old-one. Mariusz and I have been
discussing it intermittently… hence the slow follow-up.
In conclusion, I just can't see that the functionality here justifies the
additional passthrough kwarg on the `CharField`.
We can already specify `pattern` via the widget attributes, and a
validator if desired.
I'd much rather we used that API, than add an extra code path for the same
here.
In a project subclassing `CharField` to wrap your own options is likely.
As such I'm going to close this as `wontfix`. In the 10 years since this
was opened things have progressed — as you said on the PR, we don't need
the fallback — I think it was that which may have justified the ticket,
but I don't think ever in that time there's been a stable enough JS option
that we'd have wanted to include... — hence it sitting here I guess.
Even if we're not merging your fix, getting it to a conclusion is a good
step forwards, so thanks! 🎁
I hope that makes sense. C.
--
Ticket URL: <https://code.djangoproject.com/ticket/17637#comment:8>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/010701813d4babc5-6e74e8b5-324e-4c34-9142-587affb0d3e4-000000%40eu-central-1.amazonses.com.
