-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Adding unit field to USA #4235
Adding unit field to USA #4235
Conversation
This looks good - why only USA? |
data/address-formats.json
Outdated
["subdistrict"], | ||
["district"], | ||
["city"], | ||
["province", "postcode"] | ||
] | ||
}, | ||
{ | ||
"countryCodes": ["au"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can just combine this with "ca"
(Canada), they're the same
Thanks @JamesKingdom this looks great 👍 |
["housenumber", "suburb"], | ||
["city", "postcode"] | ||
] | ||
}, | ||
{ | ||
"countryCodes": ["vn"], | ||
"format": [ | ||
["housenumber", "street"], | ||
["housenumber", "street","unit"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don’t think we should include addr:unit
in the Vietnam address format. There’s only one (mistaken) use of this tag in all of Vietnam, reflecting the fact that units aren’t numbered in that country. Floor numbers are sometimes given in addresses, but generally the smallest level of addressing is the building number.
Addresses can be quite granular even without addr:unit
, because the addr:housenumber
and addr:street
tags are both hierarchical. For example, this is the address of building 8, alley 251, lane 48, Nguyễn Khang street:
Số 48/251/8 đường Nguyễn Khang
Xã Yên Hòa
Quận Cầu Giấy
Hà Nội
addr:housenumber=48/251/8
addr:street=Nguyễn Khang
addr:subdistrict=Yên Hòa
addr:district=Cầu Giấy
addr:city=Hà Nội
addr:housenumber
can be even more complex, for example 7/14/12/3/23
. On the other hand, some mappers prefer to put all the alley and lane numbers in addr:street
so the map doesn’t look so cluttered, for example lane 19, Liễu Giai street.
More information about Vietnam’s address formats:
https://vi.wikipedia.org/wiki/Địa_chỉ
https://youbianku.com/files/upu/VNM.pdf
http://vforum.vn/diendan/showthread.php?91066-Thon-Xom-Ap-Xa-Phuong-Huyen-Quan-Tinh-Thanh-Pho-trong-tieng-Anh-la-gi-khi-viet-dia-chi-nha
/ref #2252
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alright, thanks for the information, I removed it in #4242
Is it possible to leave this "addr:unit"-field away in the German version, especially in Austria? It's useless for adresses in Austria and Germany. |
I think we should remove addr:unit in the But I must say... I think it was a bad move to add |
Why? |
It is beneficial for US, CA, AU and doubtlessly other countries around the world (I guess Malaysia, HK...) But some countries already had an address format in iD, why change them?
I have my own bias about why addr:unit should not be in a standard default address format worldwide. |
For HU and SI I also do not see any benefit. |
I don't think I read the wiki page for "Unit" is very rarely used in Irish addresses, mostly for separate addresses within a (light) industrial estate. So you might see "Mappy Ltd., Unit 5, Whatever Road, Ballytown Industrial Estate, Co. Whatever". Here "Unit" is |
|
Please submit PRs so that the address field looks however you want it to look. |
Closes #3980