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

Invalid country code error #365

Open
dineshkumarab opened this issue Jan 12, 2023 · 10 comments · May be fixed by #375
Open

Invalid country code error #365

dineshkumarab opened this issue Jan 12, 2023 · 10 comments · May be fixed by #375

Comments

@dineshkumarab
Copy link

Hi,
I am getting the following error in the iOS platform. Not in Android
Error: The country code is invalid.
Code: 1

Plugin version
0.7.2

Flutter version
Flutter 3.3.2

@dineshkumarab
Copy link
Author

dineshkumarab commented Jan 12, 2023

After encountering the bug, I dig into plugin code for Android and ios. In Android, if the user has not provided any region code then It will try to find the country code by another possible solution(method maybeExtractCountryCode()). But in the iOS case, if the user has not provided the region code then it will throw an invalid country code error.

@Anton-Tesenchuk
Copy link

Hi,
I have the same issue. Only on the IOS.

@Abdelazeem777
Copy link

Same here

2 similar comments
@joshuamoreno1
Copy link

Same here

@jakubjaniec
Copy link

Same here

@ashley-arta
Copy link

Verified locally that this issue is fixed by https://github.com/natintosh/plugin_libphonenumber/pull/21/files.
We need to upgrade the libphonenumber_plugin lib to 0.3.1, and remove the deprecated getNameForNumber method.
@natintosh if you are busy, I can make open a PR for this

@ashley-arta ashley-arta linked a pull request Mar 6, 2023 that will close this issue
@ashley-arta
Copy link

Verified locally that this issue is fixed by https://github.com/natintosh/plugin_libphonenumber/pull/21/files. We need to upgrade the libphonenumber_plugin lib to 0.3.1, and remove the deprecated getNameForNumber method. @natintosh if you are busy, I can make open a PR for this

@natintosh what do you think?

@chiptech-mitchell
Copy link

Any news on getting this merged, I encountered this issue and had to add a dependency override to get around it

@ashley-arta
Copy link

@chiptech-mitchell the owner has updated the dependency in the latest release

@chiptech-mitchell
Copy link

chiptech-mitchell commented May 17, 2023

@ashley-arta Oh ok cool so it's fixed in 0.7.3+1?, that's good to hear I saw your open PR and just thought it wasn't fixed yet.

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 a pull request may close this issue.

7 participants