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

survey_date and date:survey -> survey:date #860

Merged
merged 2 commits into from Apr 12, 2023

Conversation

Marc-marc-marc
Copy link
Contributor

use more common, documented and following common namespace convention https://wiki.openstreetmap.org/wiki/Key%3Asurvey%3Adate

use more common, documented and more followinng commonn namespace convention
https://wiki.openstreetmap.org/wiki/Key%3Asurvey%3Adate
@github-actions
Copy link

🍱 Preview the tagging presets of this pull request here: https://pr-860--ideditor-presets-preview.netlify.app/id/dist/#locale=en.

Copy link
Member

@tyrasd tyrasd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The date:survey variant is currently only found 12 times. I think it is not necessary to add rules for such super rare, but possible "typo" of a tag, as this would only make the list unnecessarily hard to maintain. It's better to manually clean up such mistakes or using a dedicated tool.

+1 for survey_date: it is still pretty commonly found and listed as a deprecated version of the tag.

@tyrasd tyrasd merged commit 98bd3e3 into openstreetmap:main Apr 12, 2023
5 checks passed
@Marc-marc-marc Marc-marc-marc deleted the patch-1 branch April 19, 2023 17:16
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.

None yet

2 participants