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

Sg and update from source #2

Merged
merged 54 commits into from
Oct 24, 2019
Merged

Conversation

kishoreknowapp
Copy link
Owner

No description provided.

gvuyk and others added 30 commits April 19, 2019 15:59
Some PRs (e.g. validatorjs#1010 and validatorjs#1017) haven't been compiled
properly.
This is an internal, undocumented utility that should not be
exported.
…cale (validatorjs#1035)

* isMobilePhone: add additional prefix for KE locale

* add Chile mobile number in isMobilePhone
* Add zh-TW National ID Rule

* Add zh-TW ID rule on README.md

* fit eslint rules

* add built code

* add zh-TW id card number test

* fixed test on zh-TW id card numbers
* fix(isISO8601): add additionnals checks for leap years

* fix(isDecimal): correct decimal point type for some locales

* refactor(rtrim): implement the logic of ltrim

* fix(ltrim): add chars escaping

* test: add thrown errors handling

* test: add missing test cases

* fix(isIdentityCard): remove default value for locale

* chore: add nyc config for code coverage

* chore: add compiled bundles
* added support for IL in isIdentityCard()

* fix ci

* fix CI #2

* fix CI #3

* fix conflicts

* chnage locale to 'he-IL'
* added support for IL in isIdentityCard()

* fix ci

* fix CI #2

* fix CI #3

* fix conflicts

* chnage locale to 'he-IL'

* change regex and if statments

remove inefficient code.

* remove if statement ,its not needed.

* fix eslit problem
* validatorjs#990. Fix for Fiji mobile phones format validation.

* chore: tests and update readme for Fiji isMobile fix

closes validatorjs#991
This PR adds support for Dutch (nl-NL) locale for isMobilePhone.

closes validatorjs#923 
====
* fix(isMobilePhone): add dutch phone number format

* fix: clean-up PR validatorjs#932

- add tests
- update README
…datorjs#1055)

Co-authored-by: MiChiano <fgnm@gmail.com>

closes validatorjs#975


* Add Malta Postal Code (XXX0000) and Phone Number (+356XXYYYYYY)

* add malta (MT) to readme and test case

* add ar-MA mobile phone

* fix: update pr validatorjs#975

- clean-up merge conflicts
- add tests for postal code, ref: https://en.wikipedia.org/wiki/Postal_codes_in_Malta

* fix: remove ar-MA locale
This commit adds the Bahrain (ar-BH) locale for isMobile.
Co-authored-by: Gibbets <stevepeterka@online.de>

* Added isMobile de-AT

* Added local de-AT for isMobile

* clean up isMobile Austria number

closes validatorjs#1038 validatorjs#1028
* change hash test regex to include uppercase chars

* Clean PR on allow uppercase for isHAsh
- Co-authored-by: Ezrqn Kemboi <ezrqnkemboi@gmail.com>
- Closes validatorjs#982 validatorjs#987
ezkemboi and others added 24 commits July 25, 2019 16:23
This PR fixes ja_JP locale to support 060 prefix and also issue with the general sequence.

* fix Japanese mobile phone number

* add 060 for ja-JP of isMobile
- ensure that even when user does not pass max and min, set defaults for the same
- instance, validator.isLength('aa') returns false, instead, we should return true and set min as 0 and max as undefined
- close validatorjs#939
- when lat starts with (, it makes it a true latitude.
- Closes validatorjs#929
The (relative) link to contributors was defaulting to `/blob/master/graph/contributors`, I made it absolute instead.
* feature(isBIC): add isBIC validation
- validate valid BIC
- refers part of validatorjs#951

* chore: add on BIC definition
…rjs#1152)

* postal code validation fix for locale IN

* regex updated
* Added validation for '0x'-prefixed hex strings.

* Added 0h-prefix as an acceptable hexadecimal format.
…js#1166)

* WIP: modified ja-JP check.

* WIP: added test cases.

* Added one more invalid test case.
@kishoreknowapp kishoreknowapp merged commit 8c42278 into master Oct 24, 2019
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.