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

chore(deps): Update @sindarius/gcodeviewer (#1755) #1783

Merged
merged 1 commit into from
Feb 17, 2024

Conversation

TopoDiFogna
Copy link
Contributor

Description

Update @sindarius/gcodeviewer to latest after fix in upstream library.

First update (d34aa64) was to include OrcaSlicer in the supported slicers list; unfortunately it was including a bug where unlisted slicer features where aborting the rendering.

The new fix in the upstream library ensures that if supported slicers will add new features the render will continue to work and unsupported features will have a generic color instead of abort the render and throw an error.

Related Tickets & Documents

#1752

Sindarius/npm_gcodeviewer#13
Sindarius/npm_gcodeviewer@be58cce

Mobile & Desktop Screenshots/Recordings

No visual change

[optional] Are there any post-deployment tasks we need to perform?

No

Update @sindarius/gcodeviewer to latest after fix in upstream library.

First update (d34aa64) was to include OrcaSlicer in the supported slicers list;
unfortunately it was including a bug where unlisted slicer features where aborting the rendering.

The new fix in the upstream library ensures that if supported slicers will add new features
the render will continue to work and unsupported features will have a generic color instead of
abort the render and throw an error.

See Sindarius/npm_gcodeviewer@be58cce
@meteyou meteyou merged commit 4f46d40 into mainsail-crew:develop Feb 17, 2024
8 checks passed
@meteyou
Copy link
Member

meteyou commented Feb 17, 2024

Thx! I followed your your conversion in the Sindarius/npm_gcodeviewer already. Thanks for fixing this issue!

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

Successfully merging this pull request may close these issues.

2 participants