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

add amenity=toilet to amenity=toilets validator rule #5953

Merged
merged 1 commit into from
Feb 24, 2019
Merged

add amenity=toilet to amenity=toilets validator rule #5953

merged 1 commit into from
Feb 24, 2019

Conversation

matkoniecz
Copy link
Contributor

fixes #5952

@matkoniecz
Copy link
Contributor Author

matkoniecz commented Feb 24, 2019

https://josm.openstreetmap.de/browser/josm/trunk/data/validator/deprecated.mapcss has some other ideas

How many deprecated tags should be present to make adding a good idea? 1000? 500? 100?

@bhousel
Copy link
Member

bhousel commented Feb 24, 2019

How many deprecated tags should be present to make adding a good idea? 1000? 500? 100?

I’d prefer make tagging decisions based on improving the user experience for mappers, the needs of data consumers, and common sense, rather than picking an arbitrary number.

@quincylvania quincylvania merged commit f5d61a3 into openstreetmap:master Feb 24, 2019
@quincylvania quincylvania added this to the Next Release milestone Feb 24, 2019
@quincylvania
Copy link
Collaborator

@matkoniecz Thanks, great suggestion!

@matkoniecz
Copy link
Contributor Author

I’d prefer make tagging decisions based on improving the user experience for mappers, the needs of data consumers, and common sense, rather than picking an arbitrary number.

Obviously, all of that is important and good validator rule must match all that requirements. But I think that it makes little sense to spend time on adding rule to handle deprecated tag present 2 times worldwide and not used anymore.

@matkoniecz matkoniecz deleted the JOSM_validator_rules_import branch February 24, 2019 18:16
@bhousel
Copy link
Member

bhousel commented Feb 24, 2019

But I think that it makes little sense to spend time on adding rule to handle deprecated tag present 2 times worldwide and not used anymore.

Nice! Glad your version of “common sense” matches up with mine 😄

@matkoniecz
Copy link
Contributor Author

I just wanted to check is it matching well - for me good level for adding new rule would be 1000 uses of deprecated tag worldwide (obviously, assuming that all other requirements for deprecation rule are fullfilled).

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.

validator: replace amenity=toilet by amenity=toilets
3 participants