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

Add Malta Postal Code and Phone Number #975

Closed
wants to merge 4 commits into from
Closed

Conversation

fgnm
Copy link

@fgnm fgnm commented Feb 3, 2019

I've add Malta Postal Code (XXX0000 or XXX 0000) and Phone Number (+356XXYYYYYY). Can you please merge this as soon as possibile? It's very critical for my, many many thanks :)

@fgnm fgnm mentioned this pull request Feb 3, 2019
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.

Thanks for PR. This LGTM.

Please add tests and update the README and we should be good to go.

@profnandaa profnandaa added the 🧹 needs-update For PRs that need to be updated before landing label Apr 10, 2019
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.

Thanks for the PR! 🎉 Few more changes and we should be good with this one.

@@ -3775,6 +3775,18 @@ describe('Validators', () => {
'123456789',
'+254800723845',
],
},{
locale: 'en-mt',
Copy link
Member

Choose a reason for hiding this comment

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

should be en-MT (check case).

@@ -38,6 +38,7 @@ const patterns = {
LU: fourDigit,
LV: /^LV\-\d{4}$/,
MX: fiveDigit,
MT: /^[A-Za-z]{3}\s{0,1}\d{4}$/,
Copy link
Member

Choose a reason for hiding this comment

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

Also add tests for isPostalCode

profnandaa added a commit to profnandaa/validator.js that referenced this pull request Jul 2, 2019
profnandaa added a commit to profnandaa/validator.js that referenced this pull request Jul 2, 2019
- clean-up merge conflicts
- add tests for postal code, ref: https://en.wikipedia.org/wiki/Postal_codes_in_Malta
@profnandaa profnandaa mentioned this pull request Jul 2, 2019
@profnandaa profnandaa closed this in 20ad9b7 Jul 2, 2019
@profnandaa
Copy link
Member

@fgnm -- merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧹 needs-update For PRs that need to be updated before landing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants