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

embassy=embassy to embassy=yes (new deprecation) #341

Merged
merged 1 commit into from
Jan 10, 2022
Merged

embassy=embassy to embassy=yes (new deprecation) #341

merged 1 commit into from
Jan 10, 2022

Conversation

matkoniecz
Copy link
Contributor

@matkoniecz matkoniecz commented Jan 8, 2022

embassy=embassy is an unwanted tagging fueled by iD presets showing top taginfo suggestions

In discussion on tagging mailing list noone considered embassy=embassy as a good idea

see https://lists.openstreetmap.org/pipermail/tagging/2022-January/063459.html and say https://lists.openstreetmap.org/pipermail/tagging/2022-January/063501.html

https://taginfo.openstreetmap.org/tags/embassy=embassy#chronology

excluding embassy=embassy from taginfo suggestions also would work - but AFAIK there is no way to achieve it right now

embassy=embassy is an unwanted tagging fueled by iD presets showing top taginfo suggestions

there was confused philosophical discussion about deprecations on tagging mailing list - but of all involved noone considered embassy=embassy as a good idea

see https://lists.openstreetmap.org/pipermail/tagging/2022-January/063459.html
@matkoniecz matkoniecz changed the title embassy=embassy to embassy=yes embassy=embassy to embassy=yes (new deprecation) Jan 8, 2022
@tyrasd tyrasd changed the base branch from main to embassy-values January 10, 2022 10:30
@tyrasd tyrasd merged commit f0c6f8a into openstreetmap:embassy-values Jan 10, 2022
@tyrasd
Copy link
Member

tyrasd commented Jan 10, 2022

Thanks! FYI: I've combined this one with #343 (implementation of #340).

@matkoniecz matkoniecz deleted the patch-5 branch January 10, 2022 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants