Implementation the new Additional Method nisBR#1880

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Closed

cleitonmendonca wants to merge4 commits intojquery-validation:masterfromcleitonmendonca:master

Closed

Implementation the new Additional Method nisBR#1880

cleitonmendonca wants to merge4 commits intojquery-validation:masterfromcleitonmendonca:master

Conversation

Copy link

Contributor

cleitonmendonca commentedOct 26, 2016

Create the new addtional method nisBR for validation of NIS/PIS (Número de Identificação Social) - (Social Identification Number) for Brazillians
Closes#1879

Checklist for this pull request

Before submitting a pull request, please make sure to follow these rules:

Your code should contain tests relevant for the problem you are solving.
Your commits messages format should follow the jQuery git commit message format (http://contribute.jquery.org/commits-and-pull-requests/#commit-guidelines).
The pull request should reference existing issues or link to a reproducible demo.
Please review the guidelines for contributing (CONTRIBUTING.md) to this repository for more information.

Description

Please describe your pull request.

Thank you!

AbdullahObaidand others added4 commits

October 26, 2016 00:10

Localization: fixed typo in arabien language file.

d69e003

the message is not appearing because {0} is written (0}.Closes#1868.

Issue#1879: Alter description of method

a4eb736

Merge branch 'issue#1879'

8d201c0

Issue#1879: Alter message for user

6daede9

staabm requested changes

Dec 1, 2016

View reviewed changes

package.json

		@@ -1,60 +1,60 @@
		{
		"name": "jquery-validation",

Copy link

Member

staabmDec 1, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

I guess all the changes within this file should not be part of this PR?

Copy link

Member

staabm commentedFeb 11, 2017

Will close for now because of inactivity. Feel free to ask further questions/provide furhter information regarding this topic here. We can re-open if required.