floere / phony

E164 international phone number normalizing, splitting, formatting.
http://florianhanke.com/phony/
MIT License
1.01k stars 226 forks source link

Add 39 prefix to Brazil (MG) #386

Closed rodrigo closed 7 years ago

rodrigo commented 7 years ago

The state of Minas Gerais (Brazil) also has the 39 prefix.

See here

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 99.18% when pulling ad0d0a7c74de7383df18a64b22ad61058b11ce8b on rodrigorbltt:brazil-code-39 into 0e5eb759fc621cf2d6cd39c8d04ad26fa2633870 on floere:master.

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-0.0003%) to 99.18% when pulling ad0d0a7c74de7383df18a64b22ad61058b11ce8b on rodrigorbltt:brazil-code-39 into 0e5eb759fc621cf2d6cd39c8d04ad26fa2633870 on floere:master.

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-0.0003%) to 99.18% when pulling ad0d0a7c74de7383df18a64b22ad61058b11ce8b on rodrigorbltt:brazil-code-39 into 0e5eb759fc621cf2d6cd39c8d04ad26fa2633870 on floere:master.

wagner commented 7 years ago

This PR is invalid. NDC 39 does not exist.

floere commented 7 years ago

@wagner So is the information on the guiamais page incorrect? /cc @rodrigorbltt

rodrigo commented 7 years ago

@floere Yes, I've looked on http://www.anatel.gov.br/institucional/ and there's no record of this code as a avaliable prefix.

floere commented 7 years ago

Thanks a lot, @wagner and @rodrigorbltt ! Boa tarde! 🌤