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.

Sign up for GitHub

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

Jump to bottom

add some new surface values #597

Merged
merged 3 commits into from
Sep 29, 2022
Merged

add some new surface values #597

merged 3 commits into from
Sep 29, 2022

Conversation

matkoniecz
Copy link
Contributor

No description provided.

@matkoniecz
Copy link
Contributor Author

disclaimer: this is partially done to promote unhewn_cobblestone

As far as I see this has a general support and OSM community was asked about it and it was confirmed to be a good idea, but StreetComplete is quite intensively pushing it (for reasons described at https://wiki.openstreetmap.org/wiki/Tag%3Asurface%3Dcobblestone )

@matkoniecz
Copy link
Contributor Author

I wanted to add surface=grass_paver but I was unsure is it better to describe it as "Grass Paver" or "Grass Pavers".

tyrasd
tyrasd reviewed Sep 29, 2022
View reviewed changes
data/fields/surface.json Outdated Show resolved Hide resolved
@tyrasd tyrasd merged commit 174a15e into openstreetmap:main Sep 29, 2022
@matkoniecz matkoniecz deleted the patch-2 branch September 29, 2022 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ZeLonewolf ZeLonewolf ZeLonewolf left review comments

@tyrasd tyrasd tyrasd left review comments

Assignees
No one assigned
Labels
new-value
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@matkoniecz @tyrasd @ZeLonewolf