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

Chore clean 1034 #1085

Closed
wants to merge 4 commits into from
Closed

Conversation

ezkemboi
Copy link
Member

Clean up PR #1034.
Make sure to add test for invalid isIBAN locale.

check if the string is IBAN number,

(locale is one of ['AD','AT','BH','BE','BA','BG','CY','CZ','DE','DK','EE','FO','FI','FR','GB','GE','GI','GL','GR','HU','HR','IE','IS','IT',TN] OR defaults to 'any'. If 'any' or a falsey value is used, function will check if any of the locales match with the first two letters of the IBAN which coincide with the country Alpha-2 code name).
@ezkemboi ezkemboi force-pushed the chore-clean-1034 branch 2 times, most recently from 9d24518 to 6497f8c Compare August 11, 2019 12:50
- fix linter issues
- fix tests
- seperate unnecessary files changes
- closes validatorjs#1034 validatorjs#951
- Other references validatorjs#1071
@profnandaa
Copy link
Member

Landed in #1243 // that was my bad, I'd forgot about this PR. Thanks for the contrib through.

@profnandaa profnandaa closed this Feb 15, 2020
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.

3 participants