Skip to content

fix(XMLWriter): Bunp vtk.js version to fix crash when using XMLWriter#92

Merged
floryst merged 1 commit intoKitware:masterfrom
tamaynard:bump-vtk-version
Dec 16, 2022
Merged

fix(XMLWriter): Bunp vtk.js version to fix crash when using XMLWriter#92
floryst merged 1 commit intoKitware:masterfrom
tamaynard:bump-vtk-version

Conversation

@tamaynard
Copy link
Copy Markdown
Contributor

Getting "Uncaught TypeError: pako__WEBPACK_IMPORTED_MODULE_1__.default.deflate is not a function" when attempting to use XMLWriter. Bumping should fix this.

Copy link
Copy Markdown
Collaborator

@sankhesh sankhesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are the changes in the package-lock file intentional? They seem to be removing the original dependencies and bringing in different ones.

@tamaynard
Copy link
Copy Markdown
Contributor Author

Yes, I just followed the procedure as in 7fc434c to update vtk.js. I tested against the latest versions of node and npm and didn't find any issues.

@floryst
Copy link
Copy Markdown
Contributor

floryst commented Dec 16, 2022

I think this might be due to react-vtk-js still inadvertently bundling parts of vtk.js. In any case, the changes look fine for now, but there's still an underlying issue.

@floryst floryst merged commit a1df59e into Kitware:master Dec 16, 2022
github-actions Bot pushed a commit that referenced this pull request Dec 16, 2022
fix(XMLWriter): Bunp vtk.js version to fix crash when using XMLWriter a1df59e
@jourdain
Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 1.15.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@tamaynard
Copy link
Copy Markdown
Contributor Author

Yes it's not ideal at all. But for now this was the path of least resistance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants