X Tutup
The Wayback Machine - https://web.archive.org/web/20220319234348/https://github.com/validatorjs/validator.js/pull/1506
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

feature: Oman mobile phone validation added to isMobilePhone #1506

Merged

Conversation

dev-sna
Copy link
Contributor

@dev-sna dev-sna commented Oct 27, 2020

This pull request adds support for validation of Oman's mobile phone numbers to isMobilePhone validator.

Checklist

  • PR contains only changes related; no stray files, etc.
  • README updated (where applicable)
  • Tests written (where applicable)

@codecov
Copy link

@codecov codecov bot commented Oct 27, 2020

Codecov Report

Merging #1506 (963d0a1) into master (0b0b4e9) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #1506   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           99        99           
  Lines         1773      1773           
=========================================
  Hits          1773      1773           
Impacted Files Coverage Δ
src/lib/isMobilePhone.js 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0b0b4e9...34bd8bc. Read the comment docs.

Copy link
Member

@profnandaa profnandaa left a comment

LGTM, thanks for your contrib! 🎉

@profnandaa
Copy link
Member

@profnandaa profnandaa commented Nov 19, 2020

Pls fix the merge conflicts.

@dev-sna
Copy link
Contributor Author

@dev-sna dev-sna commented Nov 30, 2020

@profnandaa conflicts resolved.

Copy link
Member

@profnandaa profnandaa left a comment

LGTM, thanks for your contrib! 🎉

@profnandaa profnandaa merged commit 9dc4ed3 into validatorjs:master Dec 1, 2020
1 check passed
@abdulmoizeng
Copy link

@abdulmoizeng abdulmoizeng commented Dec 2, 2020

Much needed PR, Thanks @dev-sna 👍

@nasirhm
Copy link

@nasirhm nasirhm commented Dec 2, 2020

Awesome addition to the library @dev-sna 👍

@abdulmoizeng
Copy link

@abdulmoizeng abdulmoizeng commented Mar 17, 2021

Thanks for creating this, was much needed @dev-sna

@abdulmoizeng
Copy link

@abdulmoizeng abdulmoizeng commented Mar 17, 2021

@profnandaa By when is the team releasing this?

@tux-tn
Copy link
Member

@tux-tn tux-tn commented Mar 17, 2021

@abdulmoizeng a new release is planned before the end of this month

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

5 participants
X Tutup