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 outdoor shops #714

Merged
merged 7 commits into from
Apr 14, 2016
Merged

add outdoor shops #714

merged 7 commits into from
Apr 14, 2016

Conversation

nvkelso
Copy link
Member

@nvkelso nvkelso commented Apr 14, 2016

Connects with #674.

  • add outdoor shops to POIs layer
  • [test] add tests for outdoor shops
  • [migration] add outdoor shops for OSM point and polygon
  • [docs] reformat layers reference to split kind values onto individual lines for better git mojo
  • [docs] add new outdoor shop POI kinds
  • [docs] enable more href anchors for kind values and other headings

@nvkelso nvkelso added this to the v0.10.0 milestone Apr 14, 2016
This was referenced Apr 14, 2016
* `1st Order Admin Lines`, `Admin-0 country`, `Admin-1 boundary`, `Admin-1 region boundary`, `city_wall`, `country`, `county`, `Disputed (please verify)`, `fence`, `Indefinite (please verify)`, `Indeterminant frontier`, `International boundary (verify)`, `Lease limit`, `Line of control (please verify)`, `municipality`, `Overlay limit`, `retaining_wall`, `snow_fence`, `state`.
#### Boundary kind values:

* `1st Order Admin Lines`
Copy link
Member

Choose a reason for hiding this comment

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

Since #687, we updated the kinds - looks like we forgot to update the docs 😞

Copy link
Member Author

Choose a reason for hiding this comment

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

Good catch! Let's track that with: #718.

@zerebubuth
Copy link
Member

LGTM

@nvkelso nvkelso merged commit ef860e8 into master Apr 14, 2016
@nvkelso nvkelso deleted the nvkelso/outdoor-stops branch April 14, 2016 15:51
@nvkelso nvkelso removed the in review label Apr 14, 2016
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.

2 participants