Skip to content

HOTFIX - Fixed the bug on BA mobile phone regex - #1356

Merged
profnandaa merged 4 commits into
validatorjs:masterfrom
tensorspaceset:fix/update-for-ba-phone-regex
Jun 27, 2020
Merged

HOTFIX - Fixed the bug on BA mobile phone regex#1356
profnandaa merged 4 commits into
validatorjs:masterfrom
tensorspaceset:fix/update-for-ba-phone-regex

Conversation

@tensorspaceset

Copy link
Copy Markdown
Contributor

Updated the regular expression for bosnian mobile numbers by removing lookbehinds and modifying the expression to keep the right checks for these mobile numbers

@tensorspacesettensorspaceset changed the title Fixed the bug on BA mobile phone regexHOTFIX - Fixed the bug on BA mobile phone regexJun 11, 2020
@profnandaa

Copy link
Copy Markdown
Member

Can fix the merge conflict?

@profnandaaprofnandaa added 🧹 needs-update For PRs that need to be updated before landing and removed 🧹 needs-update For PRs that need to be updated before landing labels Jun 12, 2020

@profnandaaprofnandaa left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thanks!

@profnandaaprofnandaa left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Oops forgot, can you bring back the rest of the work now, I'd removed it in #1355; rebase with the current master.

@profnandaaprofnandaa added the 🧹 needs-update For PRs that need to be updated before landing label Jun 13, 2020
@profnandaa

Copy link
Copy Markdown
Member

ping @MladenZeljic

Added verification for bosnian mobile phone numbers
Fixed bs locale
Updated locale in readme
Updated the regular expression for bosnian mobile numbers by removing lookbehinds and modifying the expression to keep the right checks for these mobile numbers
@tensorspaceset
tensorspacesetforce-pushed the fix/update-for-ba-phone-regex branch from ff4007e to 1e4a05cCompareJune 27, 2020 19:14
@tensorspaceset

tensorspaceset commented Jun 27, 2020

Copy link
Copy Markdown
ContributorAuthor

@profnandaa Sorry, I've been busy. Readme, validator and regex should be here. Anything else to add?

@profnandaaprofnandaa left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@profnandaa
profnandaa merged commit 6e26cce into validatorjs:masterJun 27, 2020
@profnandaaprofnandaa mentioned this pull request Sep 18, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🧹 needs-updateFor PRs that need to be updated before landing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@tensorspaceset@profnandaa