search

joypixels / emojione-assets

[Archived] JoyPixels Emoji Assets now maintained at https://github.com/joypixels/emoji-assets.
https://www.joypixels.com
Other
65 stars 13 forks source link

CDN version 3.1.1 exceeded the configured limit of 50 MB. #31

Open lacerdev opened 7 years ago

lacerdev commented 7 years ago

When I try to access CDN of version 3.1.1 emojione-assets shows the message below:

"Package size exceeded the configured limit of 50 MB."

Eg.: https://cdn.jsdelivr.net/npm/emojione-assets@3.1.1/png/32/002a.png

This is not only true with this release and not only with the assets, but the emojione.js too. Are there any predictions for a solution to the problem with jsdelivr?

MartinKolarik commented 7 years ago

The main emojione package (version >= 3.1.2) should work fine now (emojione/emojione#534). Since the suggested solution in the linked issue was moving files to emojione-assets, I assumed everything that's useful for web is in the main package. Is that not the case?

lacerdev commented 7 years ago

https://cdn.jsdelivr.net/npm/emojione@3.1.2/lib/js/emojione.min.js is working fine, but the emojione-assets does not. The release 3.1.2 emojione-assets does not exist and the release 3.1.1 has the problem.

https://cdn.jsdelivr.net/npm/emojione-assets@3.1.1/png/32/002a.png - PROBLEM!

https://cdn.jsdelivr.net/npm/emojione-assets@3.1.0/png/32/002a.png - Works fine!

MartinKolarik commented 7 years ago

@lasdeparq I just realized we have a special endpoint for emojione assets, which is probably what you should use: https://cdn.jsdelivr.net/emojione/assets/3.1/png/64/002a.png

I have to admit I couldn't quickly find any docs for this but someone from the emojione team should be able to say if this is still the correct method and if it's documented anywhere.

lacerdev commented 7 years ago

This link worked. But it does not have the release 3.1 for emojione.js

https://cdn.jsdelivr.net/emojione/3.0/lib/js/emojione.min.js - Fine. https://cdn.jsdelivr.net/emojione/3.1/lib/js/emojione.min.js - "Couldn't find the requested release version 3.1."

MartinKolarik commented 7 years ago

https://cdn.jsdelivr.net/emojione/3.0/lib/js/emojione.min.js should be https://cdn.jsdelivr.net/npm/emojione@3.1/lib/js/emojione.min.js for versions > 3.0.

lacerdev commented 7 years ago

Ok. Tested and approved.

Just for summarize:

MartinKolarik commented 7 years ago

Can we keep this open until someone from the emojione teams confirms this is documented somewhere? As I said, I couldn't quickly find any docs for this, other than in comments to other issues where people asked about this, and that's pretty hard to find.

Alternatively, maybe consider sending a PR to explain it here: https://github.com/emojione/emojione/blob/master/INSTALLATION.md