SteamDeckHomebrew / decky-loader

A plugin loader for the Steam Deck.

https://decky.xyz

GNU General Public License v2.0

4.58k stars 158 forks source link

Add Plugin.uninstall callback support #555

Closed wheaney closed 6 months ago

wheaney commented 11 months ago

https://github.com/SteamDeckHomebrew/decky-loader/issues/536

Please tick as appropriate:

[x] I have tested this code on a steam deck or on a PC
[x] My changes generate no new errors/warnings
[x] This is a bugfix/hotfix
[ ] This is a new feature

Description

This fixes issue: #536

wheaney commented 11 months ago

Here is the PR for the plugin template update: https://github.com/SteamDeckHomebrew/decky-plugin-template/pull/34

wheaney commented 10 months ago

@TrainDoctor what's next for this?

TrainDoctor commented 10 months ago

@TrainDoctor what's next for this?

Needs testing. @SteamDeckHomebrew/decky-testers please give this a shot and let me know if you encounter any issues.

TrainDoctor commented 9 months ago

@wheaney apologies for the delay. I finally have time to give this a whirl tomorrow. If you can rebase it to latest main that'd make it much easier for me to test.

wheaney commented 9 months ago

I'm not sure what was going on with that first merge, but I think it's all sync'ed up now.

TrainDoctor commented 7 months ago

I think this is a good addition - can you remove the empty file and rebase? Will do some testing the next days.

@wheaney once this change is made I can pretty easily merge this. Thanks.

wheaney commented 7 months ago

The commit history here was confusing me, so I just rebased, removed the empty file, and force pushed.

JohnnyCrazy commented 6 months ago

Mergin, thanks for this @wheaney !