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 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

Choose a reason for hiding this comment

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

LGTM, thanks for your contrib! 🎉

@profnandaa
Copy link
Member

Pls fix the merge conflicts.

@profnandaa profnandaa added the 🧹 needs-update For PRs that need to be updated before landing label Nov 19, 2020
@dev-sna
Copy link
Contributor Author

dev-sna commented Nov 30, 2020

@profnandaa conflicts resolved.

Copy link
Member

@profnandaa profnandaa left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for your contrib! 🎉

@abdulmoizeng
Copy link

abdulmoizeng commented Dec 2, 2020

Much needed PR, Thanks @dev-sna 👍

@nasirhm
Copy link

nasirhm commented Dec 2, 2020

Awesome addition to the library @dev-sna 👍

@profnandaa profnandaa removed the 🧹 needs-update For PRs that need to be updated before landing label Dec 3, 2020
@abdulmoizeng
Copy link

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

@abdulmoizeng
Copy link

@profnandaa By when is the team releasing this?

@tux-tn
Copy link
Member

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
Development

Successfully merging this pull request may close these issues.

5 participants