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

Remove invalid option extruder #294

Merged
merged 2 commits into from
Sep 13, 2023
Merged

Remove invalid option extruder #294

merged 2 commits into from
Sep 13, 2023

Conversation

LongDono
Copy link
Contributor

@LongDono LongDono commented Sep 12, 2023

extruder is a valid option for filament_motion_sensor, but not for filament_switch_sensor

Option 'extruder' is not valid in section 'filament_switch_sensor runout_sensor'

Once the underlying issue is corrected, use the "RESTART"
command to reload the config and restart the host software.
Printer is halted

Frix-x and others added 2 commits September 12, 2023 10:23
extruder is a valid option for filament_motion_sensor, but not for filament_switch_sensor
@Frix-x
Copy link
Owner

Frix-x commented Sep 13, 2023

Oh, good catch. I'm not using one myself so didn't see this error. I'll merge to the develop branch for the next release :)

@Frix-x Frix-x changed the base branch from main to develop September 13, 2023 12:04
@Frix-x Frix-x merged commit 498f409 into Frix-x:develop Sep 13, 2023
@LongDono LongDono deleted the patch-1 branch September 13, 2023 12:53
@LongDono LongDono restored the patch-1 branch September 13, 2023 12:59
@LongDono
Copy link
Contributor Author

LongDono commented Sep 13, 2023

Happy to help but I'm new to this... Hence my deleting the branch while on mobile and then restoring it thinking that was a mistake. On the desktop, I can see it says it was safe to delete. Thank you for all your work on Klippain!

@LongDono LongDono deleted the patch-1 branch September 13, 2023 13:01
@Frix-x Frix-x mentioned this pull request Nov 17, 2023
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