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
Wrong Czech phone number prefix #34
Comments
Ahoj @igorkulman, |
Prečo proste na validaciu nie je použité štandardné riešenie ako https://github.com/iziz/libPhoneNumber-iOS ale je to tu ručne a s chybami? Videl som sťažnosti, že nefungujú aj niektoré Orange predvoľby. |
Kludne ak mas cas, tak vytvor PR proti feature branchi eKarantena a pridaj tuto lib. |
VC, ktory berie vstup na tel. cislo je tu:
|
To ako vážne? Že ja mám opravovať chyby v tejto aplikácií, ktorá určite stála milióny? Naviac v úplne hroznej code base bez jediného testu, bez ošetrenia chybových stavov atď? |
Dakujeme Igor, cenime si pristup, presne preto sme to spravili opensource. My uz bohuzial nemame cas to fixnut, dnes som bol dve fury penazi do banky odviezt a bol som si vyberat lak na moje nove lambo. To da zabrat. |
0.00€ je presna suma. |
Ahoj @igorkulman, tento projekt sme dali von ako open-source, aby sme ho sprístupnili verejnosti, ale zároveň sa neštítime pomoci zvonka. Ak chceš aktívne pomôcť zlepšiť túto aplikáciu, sme otvorení spolupráci, nie je to však must. Je mi jasné, že nie každý má čas na dobrovoľníctvo, popr nemusí súhlasiť s kvalitou codebaseu. Ďakujem za pochopenie. S pozdravom, |
Inak ta lib - https://github.com/iziz/libPhoneNumber-iOS je outdated (Swift 3), nas projekt pouziva Swift 5.2+ |
Ja aktuálne používam https://github.com/marmelroy/PhoneNumberKit. Využíva rovnaké Google data, len je to novšia Swift implementácia. Je v nej nielen samotná validacia ale aj zoznam krajín s predvoľbami na nahradenie toho jsonu v projekte. |
The phone number prefix for the Czech Republic in the app is +42 with is wrong. The correct prefix is +420.
This makes the app unusable for everyone trying to return home from the Czech Republic.
The text was updated successfully, but these errors were encountered: