Skip to content

Change underscores to hyphens in placeholder api key#147

Merged
nvkelso merged 2 commits intotilezen:masterfrom
rmglennon:master
Jun 26, 2017
Merged

Change underscores to hyphens in placeholder api key#147
nvkelso merged 2 commits intotilezen:masterfrom
rmglennon:master

Conversation

@rmglennon
Copy link
Copy Markdown
Member

@rmglennon rmglennon commented Jun 19, 2017

Part of mapzen/documentation#310

Also updates the attribution.md link to point to GitHub source after it was removed from published docs.

@rmglennon rmglennon changed the title Change underscores to hyphens in placeholder api key Do not merge yet - Change underscores to hyphens in placeholder api key Jun 19, 2017
@rmglennon rmglennon changed the title Do not merge yet - Change underscores to hyphens in placeholder api key Change underscores to hyphens in placeholder api key Jun 26, 2017
@rmglennon
Copy link
Copy Markdown
Member Author

This is ready for review and merge now.

Comment thread docs/data-sources.md
Mapzen Terrain Tiles are powered by several major open data sets and we owe a tremendous debt of gratitude to the individuals and communities which produced them.

**Attribution is required** for some of our data providers. See the [Attribution](attribution.md) document for more information.
**Attribution is required** for some data providers. See the [Attribution](https://github.com/tilezen/joerd/blob/master/docs/attribution.md) document for more information.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is strange but we talked about it IRL and agreed it was fine for now.

Copy link
Copy Markdown
Member

@nvkelso nvkelso left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@nvkelso nvkelso merged commit c8e7ca0 into tilezen:master Jun 26, 2017
@nvkelso nvkelso removed the in review label Jun 26, 2017
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