Skip to content
This repository has been archived by the owner on Dec 12, 2021. It is now read-only.

add attribute label support #102

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

add attribute label support #102

wants to merge 1 commit into from

Conversation

uiii
Copy link

@uiii uiii commented Mar 16, 2016

Hi, I've implemented support for attribute label, which when used is printed in the error messages instead of "pretiffied" attribute name (see #69 (comment)). I have updated tests, but maybe didn't cover the code completely. I didn't updated doc. Finally I've used another validation library, so I don't rely on this anymore, but I don't want throw away the work. If you found my implementation useful, here it is, if not, just delete the PR.

@ansman
Copy link
Owner

ansman commented Mar 21, 2016

Thanks, I'll take a look at this later.

michaek added a commit to michaek/validate.js that referenced this pull request Jun 10, 2016
@michaek
Copy link

michaek commented Jun 13, 2016

This seems to be an improvement on overriding the prettify function in the sense that it's scoped to the options argument of a validate() call, but it's limited to matching field names exactly to make replacements.

@thg303
Copy link

thg303 commented Nov 6, 2016

please resolve this conflict. I need this feature

@orditeck
Copy link

@ansman any update on this? Would be a nice feature

@mindaugasnakrosis
Copy link

@uiii Whst is the status?

@uiii
Copy link
Author

uiii commented Aug 18, 2020

@mindaugasnakrosis This is quite old, read the description, I think it explains it enough. I don't use this library anymore.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants