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

feat: add xk to phone data #386

Merged
merged 2 commits into from
Jun 13, 2024
Merged

feat: add xk to phone data #386

merged 2 commits into from
Jun 13, 2024

Conversation

alekangelov
Copy link
Contributor

@alekangelov alekangelov commented Jun 13, 2024

Hey, Kosovo is missing from the phone data, I added it + added a test for it. They have 6 mobile prefixes and the length of their mobile numbers is 8.

sources:

https://www.howtocallabroad.com/kosovo/
https://en.wikipedia.org/wiki/Telephone_numbers_in_Kosovo

The number format is 04x xxx xxx which is 8 digits.

Cheers

@DionIslami1
Copy link

This would be great help, great job @alekangelov 🚀🚀

@Bossa573
Copy link
Member

@alekangelov hello, please provide docs / website / wiki showing that Kosovo's mobile phone number format, thanks

@alekangelov
Copy link
Contributor Author

@Bossa573 Should I add as comment in code or just reference here? Cheers

@alekangelov
Copy link
Contributor Author

added references to code as well cheers

@Bossa573
Copy link
Member

@alekangelov the code is optional though. adding in github ticket is enough, coz it's only for manual reviewing

@alekangelov
Copy link
Contributor Author

just in case 👍

@Bossa573 Bossa573 merged commit f34cf4e into AfterShip:master Jun 13, 2024
Bossa573 added a commit that referenced this pull request Jun 13, 2024
- #386 Add Kosovo phone number format
@Bossa573
Copy link
Member

@Bossa573
Copy link
Member

@alekangelov
Copy link
Contributor Author

sick cheers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants