Revisit dependency bundling to be able to use in react projects
Upgrade deps
Fix new lint errors after dependency upgrade
Update npmignore
Fix README screenshots
Issue #, if available:
Description of changes:
After changing the bundling strategy, we were marking preact & react-redux as our deps. However, internally we alias preact/compat as react. When diagram-maker is used inside of a react project, it creates a conflict. When diagram maker calls react-redux, it calls react which needs to be aliased to preact within the diagram maker scope. The long term way forward would be:
Consider removing react-redux and use hooks instead. That would avoid the need to alias react as preact/compat in the runtime. (Note: tests still require the aliasing until we move away from enzyme).
Until we take on the above, reverting back to the old way of bundling so we can use diagram maker in react based projects.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Issue #, if available:
Description of changes: After changing the bundling strategy, we were marking preact & react-redux as our deps. However, internally we alias preact/compat as react. When diagram-maker is used inside of a react project, it creates a conflict. When diagram maker calls react-redux, it calls react which needs to be aliased to preact within the diagram maker scope. The long term way forward would be:
Until we take on the above, reverting back to the old way of bundling so we can use diagram maker in react based projects.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.