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

Added OctoPrint-TurnOffHeatOnPauseTimer plugin #1172

Merged
merged 1 commit into from
Mar 18, 2023

Conversation

Argon2000
Copy link

No description provided.

Copy link
Contributor

@jneilliii jneilliii left a comment

Choose a reason for hiding this comment

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

The only thing I'd like to warn against and ask you to change is the self._logger calls in the plugin. It would be better to change info to debug to avoid too much writing to disk. You can always just change the logging level to debug in OctoPrint's settings when you need the extra detail. Although I don't see your plugin probably doing too much of that, it would still be better that way.

@Argon2000
Copy link
Author

Should be changed now!

@Argon2000 Argon2000 requested a review from jneilliii March 18, 2023 08:30
Copy link
Contributor

@jneilliii jneilliii left a comment

Choose a reason for hiding this comment

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

One last thing I didn't catch before. Please create a release on your repo for version 0.0.1. In the future when you make updates you'll update plugin_version in setup.py and init.py and then new tag/release matching the updated version number. This is where the software update hook will catch the change and then prompt users to upgrade.

@Argon2000
Copy link
Author

Fixed!

@jneilliii jneilliii merged commit 3031088 into OctoPrint:gh-pages Mar 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants