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

CDN for leaflet-providers #95

Closed
pensierinmusica opened this issue Jun 10, 2014 · 7 comments · Fixed by cdnjs/cdnjs#3757
Closed

CDN for leaflet-providers #95

pensierinmusica opened this issue Jun 10, 2014 · 7 comments · Fixed by cdnjs/cdnjs#3757

Comments

@pensierinmusica
Copy link
Contributor

Hi,

This extension is really nice, thanks for putting it together! Is it available through any CDN? If not, shall we post it on cdnjs?

Cheers :)

@jieter
Copy link
Contributor

jieter commented Jul 15, 2014

Could be useful, why not?

@brunob @calvinmetcalf

@brunob
Copy link
Member

brunob commented Jul 15, 2014

Yes, it seems nice and it should be usefull for users.

@calvinmetcalf
Copy link
Member

go for it

@pensierinmusica
Copy link
Contributor Author

Cool... it just came to my mind that probably it's best if this is done by someone who's actively contributing to leaflet-providers, since the package would need to be updated on every major release. Anyone who's up for it? In case you're too busy I can post it on cdnjs, but wouldn't be able to track future changes. Let me know your thoughts, cheers!

@jieter
Copy link
Contributor

jieter commented Aug 20, 2014

@pensierinmusica Just made the PR, sorry for the delay.

@jieter
Copy link
Contributor

jieter commented Aug 28, 2014

It is merged. No idea how long it takes before it is actually included...

@jieter jieter closed this as completed Aug 28, 2014
@pensierinmusica
Copy link
Contributor Author

Thanks @jieter! :)

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 a pull request may close this issue.

4 participants