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

Update Form.Validator.Extras.js #1186

Closed
wants to merge 1 commit into from
Closed

Conversation

@kelexel
Copy link
Contributor

@kelexel kelexel commented Mar 5, 2013

Fix validate-match "matchName" not "working" if containing spaces
(See #1185 )

Fix validate-match "matchName" not "working" if containing spaces
@Shic1983
Copy link
Contributor

@Shic1983 Shic1983 commented Mar 7, 2013

Looks good, hopefully this will be merged soon..

@SergioCrisostomo
Copy link
Member

@SergioCrisostomo SergioCrisostomo commented Apr 27, 2014

LGTM. Specs passed.
@anutron @arian ?

@anutron
Copy link
Member

@anutron anutron commented Apr 27, 2014

there are no specs for fv.extras. can we get a jsfiddle demonstrating the issue and the fix?

@SergioCrisostomo
Copy link
Member

@SergioCrisostomo SergioCrisostomo commented Apr 29, 2014

@kelexel @NTOM could you please make a jsFiddle reproducing the problem here? Thank you!

@SergioCrisostomo
Copy link
Member

@SergioCrisostomo SergioCrisostomo commented Oct 25, 2015

Related: #1213

@anutron
Copy link
Member

@anutron anutron commented Oct 27, 2015

I guess no jsFiddle. @SergioCrisostomo I'm ok with this shipping. It looks good to me.

@SergioCrisostomo
Copy link
Member

@SergioCrisostomo SergioCrisostomo commented Oct 27, 2015

merged in 4a6cabe

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants