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

Validator::formatMessage: support dynamic error messages #218

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

Conversation

jtojnar
Copy link
Contributor

@jtojnar jtojnar commented Apr 10, 2019

This is useful when you want to show user an informative error message, whose text depends on some external data that depends on the value of some other field.

For example, my form allows user to choose a category of a team. Each category has a different limit for number of allowed members, which is specified in config.neon. I want to show the user the limit as part of the error message when she manages to exceed the limit.

  • new feature
  • BC break? no
  • doc PR: none yet

This is useful when you want to show user an informative error message, whose text
depends on some external data that depends on the value of some other field.

For example, my form allows user to choose a category of a team. Each category has a different
limit for number of allowed members, which is specified in config.neon. I want to show the user
the limit as part of the error message when she manages to exceed the limit.
@jtojnar jtojnar force-pushed the dynamic-error-message branch from b789d86 to 78032f2 Compare July 14, 2019 18:27
@dg dg force-pushed the master branch 4 times, most recently from 41ff307 to f03a237 Compare November 19, 2019 15:43
@dg dg force-pushed the master branch 2 times, most recently from 341ed9a to 97e6400 Compare December 11, 2019 19:43
@dg dg force-pushed the master branch 3 times, most recently from 9d62255 to ec59537 Compare January 2, 2020 16:48
@dg dg force-pushed the master branch 2 times, most recently from 21f7326 to 35f9b84 Compare March 9, 2020 10:42
@dg dg force-pushed the master branch 4 times, most recently from dc0cb8f to dd11397 Compare March 29, 2020 13:47
@dg dg added this to the v4.0 milestone Apr 20, 2020
@dg dg force-pushed the master branch 2 times, most recently from dd9d4ef to 458607d Compare May 15, 2020 10:07
@dg dg force-pushed the master branch 2 times, most recently from 9c1d9cf to 12d238b Compare June 5, 2020 13:20
@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