Fix distribution

[wuweiweiwu]

Reporting a Bug?

The recent changes to our dependencies #426 by @dolezel throws an error when installing.

error /Users/wuxx1045/Desktop/react-sortable-tree/example/node_modules/react-sortable-tree: Command failed.
Exit code: 127
Command: patch-package
Arguments:
Directory: /Users/wuxx1045/Desktop/react-sortable-tree/example/node_modules/react-sortable-tree
Output:
/bin/sh: patch-package: command not found

perhaps we should bundle our patched version of react-dnd-scrollzone

this was found after i released 2.4.0. For now i have set the latest tag back to 2.3.0, our previous release until this is resolved. I have also deprecated 2.4.0

[dolezel]

Sorry, my bad. That patch-package in postinstall should be needed only for dev.
https://docs.npmjs.com/misc/scripts seems prepare is right place to put the script in. Will make PR.

[wuweiweiwu]

No problem. I'm trying to test some stuff out locally with the newly built one (with the updated dependencies), but the tree isn't rendering in the demo (uses the dist/index.esm.js) but the storybook works (uses src/index.js).

I might've screwed up the rollup build. Do you mind explaining what exports: named does? as we both export a default and a named export @

[dolezel]

That is something you added to that file ;-)
2c01832#diff-ff6e5f22a9c7e66987b19c0199636480R14
I updated rollup config only with name of file in external library 9343a55#diff-ff6e5f22a9c7e66987b19c0199636480R26
Maybe it is reason why it fails now? I see you changed it to

external: [
    ...Object.keys(pkg.dependencies),
    ...Object.keys(pkg.peerDependencies),
  ],

so react-dnd/lib/DragDropContext is now missing

[wuweiweiwu]

Thanks for the help @dolezel :)

[wuweiweiwu]

released in 2.5.0.

also curious, if react-dnd-scrollzone is external in rollup, but we depend on a patched version, shouldn't we remove it from external and package it with the distribution? @dolezel

[dolezel]

True, it also need to be removed from external, 2.5.0 is not working now after fresh install 🤦‍♂️

[dolezel]

It is so tangled and so hard to test in real life. :/