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

Canadian Postal Codes #2

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Canadian Postal Codes #2

wants to merge 8 commits into from

Conversation

adambard
Copy link

So I'm pretty sure I got the formatting right. I tried to match the allCountries.txt as closely as possible. I even added tabs at the ends of lines after the fact.

I was going to remove the CA lines from allCountries.txt, but decided against it, since those lines have prettier descriptions.

@samirahmed
Copy link
Collaborator

This is awesome. However, seeing as how Canada Post is suing geocoder.ca for their reverse lookup, we will contact them first to get it cleared. Our service is technically different to geocoder.ca - we don't make Addressed into Zips, but instead return general info about them.

@adambard
Copy link
Author

I believe geocoder.ca is being sued not for their service but specifically for their free csv download (which, admittedly, I used to create the patch). Fair enough though, it's probably a good idea to check up on it.

@samirahmed
Copy link
Collaborator

Hi Adam... we contacted geocoder.ca and have decided to hold of on canadian postal codes and see how that plays out.

Canada is very highly requested so the question keeps coming up... so as soon as that is in we can pull.

@samirahmed samirahmed closed this May 4, 2012
@samirahmed samirahmed reopened this May 4, 2012
@adambard
Copy link
Author

adambard commented May 4, 2012

Quite fair, I hope it gets sorted out soon. It's weird, Canada Post is claiming copyright on the postal codes, I don't know how they have a case.

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