jseppi / Leaflet.MakiMarkers

Leaflet plugin to create map icons using Maki Icons from Mapbox.
MIT License
141 stars 39 forks source link

Provide an option to skip mandatory token #18

Open ThomasG77 opened 6 years ago

ThomasG77 commented 6 years ago

I've made a Node/Express based server (mentioned in https://github.com/jseppi/Leaflet.MakiMarkers/issues/16#issuecomment-346895728) to replace the current Mapbox Markers API.

If I want to consume the server with your plugin, I need to fork it due to the mandatory token https://github.com/jseppi/Leaflet.MakiMarkers/blob/master/Leaflet.MakiMarkers.js#L60

Can you provide in the option a way to avoid it. A parameter like skipAccessToken with default value to false could do the job. I can make a PR if you agree on the principle & the approach.

jseppi commented 6 years ago

Sure, please do submit a PR!