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

NPM package is 1.42 Mb! #82

Closed
Vadorequest opened this issue Mar 16, 2020 · 1 comment · Fixed by #104
Closed

NPM package is 1.42 Mb! #82

Vadorequest opened this issue Mar 16, 2020 · 1 comment · Fixed by #104

Comments

@Vadorequest
Copy link

https://www.npmjs.com/package/filestack-react

image

Here is what contains the package:
image

The demo folder should NOT be part of the npm package. Also, the tests aren't useful either.
Removing those 2 will decrease the package's size by around 95%.

jakubpeksa pushed a commit that referenced this issue Nov 26, 2020
…ersion, rewrite all

Completely rewrite project which implies

BREAKING CHANGE: We've removed source, file, action props, actionOptions is now called
pickerOptions, we've splitted picker component into 3 different one

fix #103, fix #82, fix #77, fix #74
jakubpeksa pushed a commit that referenced this issue Nov 26, 2020
…ersion, rewrite all

Completely rewrite project which implies

BREAKING CHANGE: We've removed source, file, action props, actionOptions is now called
pickerOptions, we've splitted picker component into 3 different one

fix #103, fix #82, fix #77, fix #74
jakubpeksa pushed a commit that referenced this issue Nov 26, 2020
…ersion, rewrite all

Completely rewrite project which implies

BREAKING CHANGE: We've removed source, file, action props, actionOptions is now called
pickerOptions, we've splitted picker component into 3 different one

fix #103, fix #82, fix #77, fix #74
jakubpeksa pushed a commit that referenced this issue Nov 26, 2020
…ersion, rewrite all

Completely rewrite project which implies

BREAKING CHANGE: We've removed source, file, action props, actionOptions is now called
pickerOptions, we've splitted picker component into 3 different one

fix #103, fix #82, fix #77, fix #74
@Vadorequest
Copy link
Author

@pcholuj Saw this on bundlephobia. (tree shacking)
Don't know if it's an issue.

https://bundlephobia.com/[email protected]

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants