decky-plugin-database icon indicating copy to clipboard operation
decky-plugin-database copied to clipboard

Update Audio Loader to 1.6.0

Open EMERALD0874 opened this issue 1 year ago • 4 comments

Audio Loader

Audio Loader is a tool for changing existing UI sounds and adding music while navigating Steam.

Unfortunately, due to recent breakages including one caused by Decky now running in desktop mode, we do not have the time or knowledge to solve bugs with the music part of Audio Loader. We have decided to make it a legacy feature until it is fixed, requiring users to check a box saying they acknowledge bugs may not be fixed and to use our GitHub for support.

Checklist:

Developer Checklist

  • [x] I am the original author or an authorized maintainer of this plugin.
  • [x] I have abided by the licenses of the libraries I am utilizing, including attaching license notices where appropriate.

Plugin Checklist

  • [x] I have verified that my plugin works properly on the Stable and Beta update channels of SteamOS.
  • [x] I have verified my plugin is unique or alternatively provides more/alternative functionality to a similar plugin already on the store.

Plugin Backend Checklist

  • No: I am using a custom backend other than Python.
  • No: I am using a tool or software from a 3rd party FOSS project that does not have it's dependencies statically linked.
  • No: I am using a custom binary that has all of it's dependencies statically linked.

Testing

  • [ ] Tested on SteamOS Stable/Beta Update Channel.

  • [x] Tested on SteamOS Preview Update Channel.

EMERALD0874 avatar Sep 14 '24 17:09 EMERALD0874

@EMERALD0874 tested on steam deck oled, under the steam preview branch. Works as expected.

I do have 2 oddities to report. First, there's no header after enabling legacy features. A new unlabeled section pops up. This may be confusing to some users.

Second, the store still shows music submissions. You can download music packs while legacy features is disabled.

suchmememanyskill avatar Sep 14 '24 18:09 suchmememanyskill

@suchmememanyskill

  1. The reason there's no label is it seemed odd to me to have "Legacy Features" listed right below the enable toggle. I think the separation is clear enough given it falls beneath the toggle and only shows up when the relevant feature set is enabled, but I can change that if necessary.
  2. Managing music packs doesn't require legacy features to be enabled as otherwise users would have to go to desktop mode to modify their downloaded packs if they couldn't figure out that they had to enable the setting. I also don't want to create a precedent that legacy features require overriding the default store view, as I think it could get messy.

EMERALD0874 avatar Sep 14 '24 18:09 EMERALD0874

  1. I meant more a label of 'music', but fair enough.
  2. Fair, i forgot this isn't cssloader and the management and store screen are the same

suchmememanyskill avatar Sep 14 '24 18:09 suchmememanyskill

Plugin fails to build in CI: https://github.com/SteamDeckHomebrew/decky-plugin-database/actions/runs/10864252954/job/30149318978

@TrainDoctor It looks like it worked here unless I'm missing something. This was build 2530 and you linked build 2529 (before I made a fix for the PNPM issue). https://github.com/SteamDeckHomebrew/decky-plugin-database/actions/runs/10864321414/job/30149465108

EMERALD0874 avatar Sep 25 '24 23:09 EMERALD0874