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

Deprecate outdoor_seating=streetside|street_side (-> outdoor_seating=terrace|sidewalk|parklet|...) #1112

Closed
danieldegroot2 opened this issue Feb 8, 2024 · 3 comments

Comments

@danieldegroot2
Copy link
Contributor

danieldegroot2 commented Feb 8, 2024

Deprecated OSM Tag(s)

outdoor_seating=streetside|street_side

Replacement OSM Tag(s)

outdoor_seating=terrace|sidewalk|parklet|...

Link to OSM Wiki page

https://wiki.openstreetmap.org/wiki/Key%3Aoutdoor_seating

Further Information

Deprecate outdoor_seating=streetside|street_side in favour of more specific value.
(though arguably could be located on a 'road lane' or 'parking space', here it may be considered ambiguous.);
I am leaving uncommon values as-is, like pedestrian_zone, street, etc.
alternatively, deprecate streetside to sync with parking value, see #1111.

Deprecation is not documented (yet). I will personally not document it, as I focus on mapping.

@tyrasd
Copy link
Member

tyrasd commented Feb 14, 2024

Hmm, a deprecation cannot be defined if there is a list of possible values to be considered in the output. This is because the tag upgrade routine in iD would not know which specific value to use. Maybe this better suited as a maproulette challenge instead?!

@tyrasd tyrasd added the waitfor-info something is unclear with the suggestion, more information is needed to evaluate this feature reques label Feb 14, 2024
@tyrasd
Copy link
Member

tyrasd commented Feb 14, 2024

See also ideditor/schema-builder#138 for a possible enhancement to support this kind of deprecation in iD as well.

@danieldegroot2
Copy link
Contributor Author

Closing as redundant for now.

@tyrasd tyrasd removed the waitfor-info something is unclear with the suggestion, more information is needed to evaluate this feature reques label Feb 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants