Skip to content

Added plugin cvprinting#1365

Merged
jneilliii merged 4 commits into
OctoPrint:gh-pagesfrom
Spini11:gh-pages
Apr 21, 2025
Merged

Added plugin cvprinting#1365
jneilliii merged 4 commits into
OctoPrint:gh-pagesfrom
Spini11:gh-pages

Conversation

@Spini11
Copy link
Copy Markdown
Contributor

@Spini11 Spini11 commented Apr 19, 2025

No description provided.

@Spini11
Copy link
Copy Markdown
Contributor Author

Spini11 commented Apr 19, 2025

Could I ask someone to verify I've added hook for updates correctly into my code? It's at the end of the init file, thank you.
https://github.com/Spini11/OctoPrint-Cvprinting/blob/master/octoprint_cvprinting/__init__.py

Copy link
Copy Markdown
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.

https://github.com/Spini11/OctoPrint-Cvprinting/blob/ee3e29baa267b2ad7d02f84a2004bccbec6ff63f/octoprint_cvprinting/__init__.py#L49: should use https://docs.octoprint.org/en/master/modules/plugin.html#octoprint.plugin.types.OctoPrintPlugin.get_plugin_data_folder.

In your software update check change updateplugindemo to cvprinting.

I think there's a lot you can do to simplify your settings, you don't need to create your own blueprint for that if you bind properly in knockout. When I have some more time and not looking from my phone I'll give proper review and suggest changes related to that.

@Spini11
Copy link
Copy Markdown
Contributor Author

Spini11 commented Apr 20, 2025

Will check the the first two things mentioned. The blueprints for updating settings and getting them is for changing it externally, not for octoprint UI. I could remove it for now from the master branch for now at least though if needed.
Thank you

Copy link
Copy Markdown
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 blueprints for updating settings and getting them is for changing it externally, not for octoprint UI

Oh, that makes sense then.

I noticed one other thing with your software update check. the pip property would work the way you have it, but would technically download the master branch every new tag release. It might be preferable to use the pattern https://github.com/Spini11/OctoPrint-Cvprinting/archive/{target_version}.zip instead.

That should do it, those changes and the other one I flagged for cloud attributes.

Comment thread _plugins/cvprinting.md Outdated
@Spini11
Copy link
Copy Markdown
Contributor Author

Spini11 commented Apr 21, 2025

Thank you, I've addressed both issues and created new release containing those updates.

@jneilliii jneilliii merged commit 2d6b71b into OctoPrint:gh-pages Apr 21, 2025
4 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in OctoPrint Backlog Apr 21, 2025
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.

3 participants