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

Import the key "golf=" as polygon features, with exceptions for golf=hole, path and cartpath #4017

Merged
merged 2 commits into from
Feb 6, 2020

Conversation

jeisenbe
Copy link
Collaborator

Related to #3611 and #661

Changes proposed in this pull request:

  • Add golf key to list of polygon keys in openstreetmap-carto.lua so that closed ways with this key will be imported as polygons, except for golf=hole, golf=path and golf=cartpath, which are always mapped as lines

While there is some debate in #661 whether rendering golf=* features in this style is appropriate, some other styles based on this one are already rendering golf=* features. Adding this key to properly import the relevant features will make it easier to adapt this stylesheet in the future.

See documentation of golf=* values at https://wiki.openstreetmap.org/wiki/Key:golf

Add golf key to list of polygon keys in openstreetmap-carto.lua so that closed ways with this key will be imported as polygons, except for golf=hole, golf=path and golf=cartpath which are always mapped as lines
@jeisenbe jeisenbe added the lua label Jan 30, 2020
@jeisenbe jeisenbe changed the title Import the key "golf=" as a polygon features, with exceptions for hole, path and cartpath Import the key "golf=" as polygon features, with exceptions for golf=hole, path and cartpath Jan 30, 2020
@jeisenbe
Copy link
Collaborator Author

jeisenbe commented Feb 6, 2020

Fixed the merge conflict

@jeisenbe jeisenbe merged commit c6a34b7 into gravitystorm:schema_changes Feb 6, 2020
@jeisenbe jeisenbe deleted the schema-golf branch February 6, 2020 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants