Opened 8 years ago
Closed 8 years ago
#8478 closed (duplicate)
Change Canadian postal code for Newfoundland
| Reported by: | Doug Harris | Owned by: | Doug Harris |
|---|---|---|---|
| Component: | contrib.localflavor | Version: | master |
| Severity: | Keywords: | ||
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | yes |
| Easy pickings: | UI/UX: |
Description
django.contrib.localflavor.ca.ca_provinces has "NF" as the postal code for Newfoundland and Labrador. According to the Canada Post web site, the proper abbreviation is "NL".
The wikipedia article on Canadian subnational postal abbreviations shows "NF" as a former abbreviation.
Attachments (1)
Change History (5)
Changed 8 years ago by
comment:1 Changed 8 years ago by
| Has patch: | set |
|---|---|
| Needs documentation: | unset |
| Needs tests: | unset |
| Patch needs improvement: | unset |
| Triage Stage: | Unreviewed → Accepted |
comment:2 Changed 8 years ago by
| Patch needs improvement: | set |
|---|
This needs a better patch. The code itself should refer to NL in the PROVINCES_NORMALIZED (including recognizing the old "nf" and returning "NL"). Also, there should be changes to tests/regressiontests/forms/localflavor/ca.py
I will work on this, but I'm posting this comment to make sure that these items are considered before this ticket is closed.
comment:3 Changed 8 years ago by
| Owner: | changed from nobody to Doug Harris |
|---|
You're 100% right. I'm assigning you to this ticket, since I'll no longer work on it.
comment:4 Changed 8 years ago by
| Resolution: | → duplicate |
|---|---|
| Status: | new → closed |
Closing as a duplicate of #10365, since that ticket contains the comment of why this isn't even necessarily a bug and why changing it is probably a bad idea.
django.contrib.localflavor.ca.ca_provinces 'NF' -> 'NL', trailing spaces removed