Update placeholder text, few links, typos#404
Conversation
|
This is ready for review and merge. You can look at the raw markdown to verify changes, or check out the preview if you want to see what happens. Expected behavior: When you are not signed in, or are signed in but have no API keys, you will continue to see placeholder text ("your-mapzen-api-key")...more enhancements coming to the placeholder text. Preview (let me know if you need credentials): |
|
Changing this back to DO NOT MERGE YET status. Results of testing are inconsistent. |
|
After discussion with @louh, we should go ahead and review and merge this PR to complete the work on the content side of this project. Remaining issues are with the code and are being tracked in mapzen/documentation#312. |
|
These changes look nice! |
These changes are part of mapzen/documentation#310
This updates the placeholder text to use your-mapzen-api-key (please make sure I didn't accidentally remove required quotation marks or anything like that) so it can be automatically replaced when signed in.
I also deleted some text that describes how to use the placeholder text (like replace it with your real key) because the key is already replaced when signed in, and the pop-up over the text will be the call to action for users who are not signed in.
There were also a few minor style (change "we" to you or third-party) and typo issues that I happened to see.