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

clarify terms of basemap use #836

Merged
merged 2 commits into from Sep 7, 2016
Merged

clarify terms of basemap use #836

merged 2 commits into from Sep 7, 2016

Conversation

jgravois
Copy link
Contributor

@jgravois jgravois commented Sep 1, 2016

No description provided.

@@ -120,7 +121,7 @@ http://esri.github.io/esri-leaflet/plugins/)
* [I want to contribute. How can I help?](https://github.com//Esri/esri-leaflet/wiki/FAQ#i-want-to-contribute-how-can-i-help)
* [I built something with Esri Leaflet can I show you?](https://github.com//Esri/esri-leaflet/wiki/FAQ#i-built-something-with-esri-leaflet-can-i-show-you)
* [I built a reusable component (layer type, api wrapper, ui control etc...) can I contribute it to Esri Leaflet?](https://github.com//Esri/esri-leaflet/wiki/FAQ#i-built-a-reusable-component-layer-type-api-wrapper-ui-control-etc-can-i-contribute-it-to-esri-leaflet)
* [What are the terms of use for Esri map tiles?](https://github.com//Esri/esri-leaflet/wiki/FAQ#what-are-the-terms-of-use-for-esri-map-tiles)
* [What are the terms of use for Esri map tiles?](#terms)
Copy link
Contributor

Choose a reason for hiding this comment

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

May want to use the term "basemaps" here to cover all types of different types?

@alaframboise
Copy link
Contributor

Only other thing I can think of is to add something in the first few lines:

Leaflet plugins for working with a handful of the most popular ArcGIS Service types. This includes Esri basemaps and feature services, as well as tiled map, dynamic map and image services. Please be sure to visit the Esri Terms of Use and attribution requirements.

@jgravois
Copy link
Contributor Author

jgravois commented Sep 1, 2016

thanks AL. i've incorporated your feedback.

better

mention limit too

incorporate als feedback

tweaks
@jgravois jgravois merged commit a70460c into Esri:master Sep 7, 2016
@jgravois jgravois deleted the terms branch September 7, 2016 23:13
jgravois added a commit to jgravois/esri-leaflet that referenced this pull request Apr 23, 2022
* clarify terms of basemap use

better

mention limit too

incorporate als feedback

tweaks

* tweak/bump faq
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.

None yet

2 participants