Skip to content

Conversation

ngross71
Copy link
Owner

No description provided.

@ngross71 ngross71 requested a review from kplotnik May 11, 2022 14:48
README.md Outdated
@@ -1,6 +1,7 @@
# Maps API for JavaScript

This repository holds a series of JavaScript based examples using the **HERE Maps API for JavaScript**. More information about the API can be found on [developer.here.com](https://developer.here.com/develop/javascript-api) under the JavaScript APIs section.
Download all files in the GitHub folder linked to the "Fork on GitHub" button on the top of the example page. Additionally, download the **test-credentials.js** file, the **template.css** file and the **meta.json** file from the folder **maps-api-for-javascript-examples** and place it in a folder above your folder in which you placed the example.
Copy link
Collaborator

Choose a reason for hiding this comment

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

This Readme is available on gihub only. Why do we send the user back to developer.here.com? Besides, I presume, that we talk to developers here who understand what is Git - so I'd expect to see here the instructions in the spirit clone the repository, host it on your local server (or npx serve) and replace credentials

@kplotnik
Copy link
Collaborator

Looks good to me, thank you.

@ngross71 ngross71 merged commit c729e87 into master May 17, 2022
@ngross71
Copy link
Owner Author

@kplotnik unfortunately I don't have rights to merge to the master repo. Can you merge my commit from my fork to the master repo? Thank you

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