-
Notifications
You must be signed in to change notification settings - Fork 39
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
**Thumbnails broken? READ THIS first** #227
Comments
No thumbnail by saving the print history but in the log is no error `2023-11-01 20:24:32,886 - INFO - ----- Start grab Image/thumbnail... ----- But an error in Octoprint log 2023-11-01 19:36:48,976 - octoprint - ERROR - Exception on /plugin/PrintJobHistory/printJobSnapshot/20231101-162600.jpg [GET] |
I fix the error in correct line 443 in PrintJobHistoryAPI.py to |
thanks worked for me as well.. |
Will an update be coming anytime soon? |
anyone who sees this and the PRs mentioned above haven't been merged yet, please head over to this issue and comment on it to try and convince OctoPrint that this plugin has been abandoned. Cheers! OctoPrint/plugins.octoprint.org#1222 /cc @Filmman586 |
I'm glad it worked for you! Did you happen to see the change in this PR that changes the cache_timeout parameter to max_age? Just curious if that would've worked as well. Sounds like the change you made is an alternate approach to the same bug. |
I have no way of editing the file, can I get the file already fixed? |
|
Thank you! I did I’ll give it a try!
…Sent from my iPhone 6 Plus
On Nov 26, 2023, at 12:23 PM, Hottabych- ***@***.***> wrote:
I have no way of editing the file, can I get the file already fixed?
Don't understand, because if you can exchange the file then you can edit your local file too e.g. with nano oder vi. As you can read in several sources it is a very simple fix. Look at #218 (comment)<#218 (comment)> alternativ #226<#226>
—
Reply to this email directly, view it on GitHub<#227 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AT3LHWTTVQS3EZC5SM5RNWDYGN3JRAVCNFSM6AAAAAA6H2OTMKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRWHA2DAOBXGQ>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
WE DID IT - i now control this plugin and have pushed the update. thanks everyone! |
#226 #214
Check out the fixes in the PR(s) (226 includes 214). The author of this repo doesn't seem to pay attention anymore but you can update the files manually per the steps I listed on those.
If they don't solve your issues, feel free to post a comment here and include the Technical Log from the print job history job having issues and I'll do my best to help.
The text was updated successfully, but these errors were encountered: