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 SpoolManager #1324

Merged
merged 3 commits into from
Sep 13, 2024
Merged

Adopt SpoolManager #1324

merged 3 commits into from
Sep 13, 2024

Conversation

WildRikku
Copy link
Contributor

Two bugfixes created, readme etc updated, release 1.7.4 created.

@jneilliii
Copy link
Contributor

Small correction needed on the fork. The software update hook needs to be updated to point to your fork/github username, etc. I would re-create the 1.7.4 tag and release after making this change. Otherwise, future updates won't get picked up by users.

https://github.com/WildRikku/OctoPrint-SpoolManager/blob/dd5189cf82f18e95240166fa8f9e13ae3e4888d2/octoprint_SpoolManager/__init__.py#L881-L918

@WildRikku
Copy link
Contributor Author

WildRikku commented Sep 12, 2024

Small correction needed on the fork. The software update hook needs to be updated to point to your fork/github username, etc. I would re-create the 1.7.4 tag and release after making this change. Otherwise, future updates won't get picked up by users.

https://github.com/WildRikku/OctoPrint-SpoolManager/blob/dd5189cf82f18e95240166fa8f9e13ae3e4888d2/octoprint_SpoolManager/__init__.py#L881-L918

Ah, one more thing where that needs to be set. Thanks for pointing me to the right place, I updated it now and updated the release, too.

I also deactivated the prerelease channel / branch as the configuration seemed wrong to me anyway (or I did not understand it). Now that I think about it, I wonder what happens to users having that channel active for the plugin? Will they switch to the stable channel? If they might end up with no channel at all, I'd rather activate it again.

Edit: From briefly looking at the source of the softwareupdate plugin, it seems users would automatically get the stable updates.

@WildRikku
Copy link
Contributor Author

Since I forgot to reference the adoption issue in my commits, I'll link the version diff here as it includes mainly changes for the plugin adoption. Maybe it helps someone in the future to take care all the things. WildRikku/OctoPrint-SpoolManager@WildRikku:OctoPrint-SpoolManager:1.7.3...1.7.4

@jneilliii
Copy link
Contributor

Edit: From briefly looking at the source of the softwareupdate plugin, it seems users would automatically get the stable updates.

yes, because all the channels included the main branch, then when the release was put out on main they would get notified of update.

OliisGit used teh CI to generate the main.zip and attach to the release assets in order for the badge counting to work on the GitHub Repo. It's way simpler to just link to the automatically generated release assets, but they don't get counted by download stats on GitHub. Here's a super simple example of what I typically do since I don't use badges.

https://github.com/jneilliii/OctoPrint-JogWheelControl/blob/c3c1c57f6db5931ef596d4e688cadb7905f8fc45/octoprint_jogwheel/__init__.py#L30-L45

Here's more information related to what OllisGit was doing: https://github.com/OllisGit/OctoPrint-KnowledgeBase/wiki/DownloadCounter

@jneilliii jneilliii merged commit f5ce41c into OctoPrint:gh-pages Sep 13, 2024
5 checks passed
@WildRikku
Copy link
Contributor Author

Edit: From briefly looking at the source of the softwareupdate plugin, it seems users would automatically get the stable updates.

yes, because all the channels included the main branch, then when the release was put out on main they would get notified of update.

OliisGit used teh CI to generate the main.zip and attach to the release assets in order for the badge counting to work on the GitHub Repo. It's way simpler to just link to the automatically generated release assets, but they don't get counted by download stats on GitHub. Here's a super simple example of what I typically do since I don't use badges.

https://github.com/jneilliii/OctoPrint-JogWheelControl/blob/c3c1c57f6db5931ef596d4e688cadb7905f8fc45/octoprint_jogwheel/__init__.py#L30-L45

Here's more information related to what OllisGit was doing: https://github.com/OllisGit/OctoPrint-KnowledgeBase/wiki/DownloadCounter

Thank you so much! For now I just generated the zip file manually and created a release like Olli and Doug did, but I noticed the Github configuration files. I will check them out and your suggestions, too :)

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