react-json-tree icon indicating copy to clipboard operation
react-json-tree copied to clipboard

Library request in cdnjs

Open extend1994 opened this issue 8 years ago • 7 comments
trafficstars

Hi, There is a library request in cdnjs. To contain the library correctly, I would like to check the files needed to be added.

The files I plan to add is: all files in /src ( including /src/utils and /src/themes )

Are they complete for users? Or other files required? Or some files not required?

Thank you for your great work!

extend1994 avatar Nov 29 '16 16:11 extend1994

I'm not sure how exactly it should work, I'm not that familiar with cdnjs. Shouldn't it be a prebuilt bundle? Otherwise - yes, /src contains everything required.

alexkuz avatar Nov 30 '16 16:11 alexkuz

cdnjs contains files running on browsers. So whether files in format like JSON___Node.js in /src may be for nodejs only but not the browsers? If there are other files not required, please tell me.

Thank you for your reponse :)

extend1994 avatar Dec 04 '16 18:12 extend1994

cdnjs contains files running on browsers.

Yeah, I get that, but I'm not sure how it will should be built to work with external React etc. All files are required, it just supposed to be built with a bundler like webpack or browserify. I think you can use a config to build it like this one:

https://github.com/christianalfoni/formsy-react/blob/master/webpack.production.config.js

alexkuz avatar Dec 05 '16 10:12 alexkuz

All files are required, it just supposed to be built with a bundler like webpack or browserify.

Agree, sometimes we just got library hosting request from users don't really know how the libs work, I'm not writing react apps, but I also wonder if this host could help, or is this the proper way to host it.

@alexkuz if you are also not sure, and not going to provide pre-bundled dist files, I think maybe we should just leave it alone, until we know about how to handle it, what do you think?

PeterDaveHello avatar Dec 22 '16 11:12 PeterDaveHello

True. It's probably not hard to build that dist (and there are probably other react libs in cdnjs already, just like that formsy-react), but I don't have much time to dig in it right now.

alexkuz avatar Dec 22 '16 16:12 alexkuz

Maybe we could just leave this issue open for a while, and close it if we can't make it after months, thanks @alexkuz

PeterDaveHello avatar Dec 23 '16 15:12 PeterDaveHello

Yep, that's fine for me

alexkuz avatar Dec 23 '16 18:12 alexkuz