Skip to content

Adding UA (Ukraine) postal code validation#947

Merged
chriso merged 2 commits into
validatorjs:masterfrom
Maxim-Mazurok:ua-postal-code
Dec 23, 2018
Merged

Adding UA (Ukraine) postal code validation#947
chriso merged 2 commits into
validatorjs:masterfrom
Maxim-Mazurok:ua-postal-code

Conversation

@Maxim-Mazurok

Copy link
Copy Markdown
Contributor

Added UA (Ukraine) postal code validation to resolve #946 issue.
Updated tests, browser build, and readme.
Rule added in according with information of Ukrainian postal codes

@Maxim-Mazurok

Copy link
Copy Markdown
Contributor Author

@chriso can you take a look, please?

@profnandaa

Copy link
Copy Markdown
Member

Thanks for your PR @Maxim-Mazurok, I'll review soon.

@Maxim-Mazurok

Maxim-Mazurok commented Dec 10, 2018

Copy link
Copy Markdown
Contributor Author

@profnandaa thank you, it would be great!

@profnandaa profnandaa 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.

@profnandaa profnandaa added the ready-to-land For PRs that are reviewed and ready to be landed label Dec 19, 2018
@Maxim-Mazurok

Copy link
Copy Markdown
Contributor Author

Thanks, @profnandaa, I linked this wiki page in original description of PR and I'm from Ukraine originally, so I can confirm that it's valid check.

@chriso

chriso commented Dec 23, 2018

Copy link
Copy Markdown
Collaborator

Thanks for the PR 😄

@chriso chriso merged commit 52f7673 into validatorjs:master Dec 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-land For PRs that are reviewed and ready to be landed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add UA postal code validation

3 participants