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

For number type, use a user-defined validation text instead of "Please enter a valid value." #138

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Piskvor
Copy link

@Piskvor Piskvor commented Nov 12, 2016

The JS library always uses a hardcoded error text: "Please enter a valid value." if a type="number" field is misfilled.
It should only fall back to the hardcoded text if no error message has been defined.

  • bugfix
  • issues: none
  • documentation: not needed
  • BC break: no

The JS library always uses a hardcoded error text if a type="number" field is misfilled.
It should only fall back to the hardcoded text if no error message has been defined.
@dg dg force-pushed the master branch 2 times, most recently from 5f19501 to 105a1af Compare January 4, 2017 14:16
@dg dg force-pushed the master branch 14 times, most recently from a6ec22a to c2c4dc3 Compare January 30, 2017 00:54
@dg dg force-pushed the master branch 2 times, most recently from 454e643 to a8c2e65 Compare February 2, 2017 03:03
@dg dg force-pushed the master branch 2 times, most recently from 571b36c to 841e5de Compare March 29, 2017 16:48
@dg dg force-pushed the master branch 4 times, most recently from 8ed13d7 to d6aa86e Compare July 14, 2017 10:58
@dg dg force-pushed the master branch 5 times, most recently from f1a8ed2 to b10e0c0 Compare July 24, 2017 14:08
@dg dg force-pushed the master branch 4 times, most recently from 49c38f3 to c6f98c1 Compare April 1, 2024 01:05
@dg dg force-pushed the master branch 4 times, most recently from 337f707 to 2146e59 Compare May 2, 2024 10:56
@dg dg force-pushed the master branch 6 times, most recently from 94f43f0 to fe374c5 Compare May 12, 2024 13:07
@dg dg force-pushed the master branch 7 times, most recently from cb844f0 to 1d8a434 Compare August 8, 2024 14:36
@dg dg force-pushed the master branch 2 times, most recently from 479185b to 8da5796 Compare October 22, 2024 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants