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

Adopt OctoLight #1229

Merged
merged 6 commits into from
Sep 16, 2023
Merged

Adopt OctoLight #1229

merged 6 commits into from
Sep 16, 2023

Conversation

thomst08
Copy link
Contributor

New maintainer of OctoLight

Closes: #1225

An issue was created on the original OctoLight repo to check if it was abandoned. I showed interest in maintaining this plugin, @gigibu5 was happy for me to take over.
gigibu5/OctoLight#41

Please let me know if I have done anything incorrectly or need to update anything else.

- Updated octolight plugin details
- Updated OctoLight images
- Added note about OctoLight - changing maintainer
- Added note about OctoLight
- Corrected date mistake in the octolight.md file
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.

couple of small tweaks necessary. I notice heavy usage of self._logger.info in your code. might recommend most those be switched to self._logger.debug to avoid writing too much to the octoprint.log.

_plugins/octolight.md Outdated Show resolved Hide resolved
_plugins/octolight.md Outdated Show resolved Hide resolved
_plugins/octolight.md Outdated Show resolved Hide resolved
- Corrected authors
- Corrected image path
@thomst08
Copy link
Contributor Author

Thank you regarding the point about the logs, I will go ahead and update the code shortly :)

@jneilliii jneilliii merged commit 1fceaf0 into OctoPrint:gh-pages Sep 16, 2023
4 checks passed
@ArtSpaziano
Copy link

Is it possible to add another Event option to have the light go on when a filament change occurs?

@thomst08
Copy link
Contributor Author

Hey @ArtSpaziano,

Submit an issue on my repo and I can look into it, we can discuss it more there.

https://github.com/thomst08/OctoLight

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

Successfully merging this pull request may close these issues.

[Abandoned Plugin] OctoLight
3 participants