search

codehz / DeckScreenshotManager

Add screenshot manager because valve don't do it. https://steamcommunity.com/app/1675200/discussions/2/6117591738155117046/
Other
3 stars 0 forks source link

Latest Steam Stable broke the plugin partly #3

Open pchristod opened 8 months ago

pchristod commented 8 months ago

Hey @codehz, hope you are doing well. I just wanted to inform you that one of the latest Steam Stable Updates broke some functionality of your wonderful Plugin. It's actually one of the most useful Plugins on the store and would love to see it back working fully. The issue I'm seeing is if you take a screenshot it will notify via "Screenshot taken" message, then throw a seconds message with an error. Unbenannt3 Additionally the Upload Counter will not increase, however the upload of the screenshot still works. I can't read the error message in full, however there's been a few plugins which broke with the latest Stable Update and it seems to be related to DFL. Maybe you can take a look at it?

Thanks!

codehz commented 8 months ago

I found the problem is not about DFL update, instead, about steam's internal 'ReactQueryClient' object which I used to clear cache for builtin screenshot manager(so the upload state can be reflect to the interface), and they just removed it...

I added a try catch to hide this error for now, I will try to figure new method

pchristod commented 8 months ago

Thanks, sounds great! That would also explain why everything still works apart from the counter.

codehz commented 8 months ago

I've already submit update to repo https://github.com/SteamDeckHomebrew/decky-plugin-database/pull/585