Babelify @jellyfin/libass-wasm #4356
Conversation
|
Kudos, SonarCloud Quality Gate passed!
|
|
note that this doesn't transpile the workers, also with this change libass-wasm should be excluded from |
|
Heya, I've ran into problems after the update of Jellyfin(-web) to 10.8.10, had no idea what was going on because Jellyfin worked on my phone and in my browser. However, it didn't load on my LG TV that runs WebOS. So without being sure about anything I opened this ticket jellyfin/jellyfin-webos#154 and rolled my system back to a version prior. Now when I had some time over I tried to roll back only jellyfin to 10.8.9 (not jellyfin-web, it was still on 10.8.10) or anything else and it was still broken so I assumed it was something else in the system, maybe nginx or something. Some hours later of bisecting nixpkgs/nixos I found that the breakage were due to jellyfin-web and not jellyfin. So I spent some time bisecting jellyfin-web and trying out specific commits, bisect log available here: jellyfin/jellyfin-webos#154 (comment) So... yeah, that pointed me here. This PR broke Jellyfin-web on LG WebOS 5 TV's. Not sure why though, I'm not sure if I have a way to inspect the browser or so. I don't even have developer tools on it enabled... 🙂 |
This is a workaround for this issue: jellyfin/jellyfin-webos#154 which is a regression from jellyfin/jellyfin-web#4356
Changes
Issues
libass-wasmmust be transpiled because of the forgottenconst. This will also be helpful when migrating to ES6.jellyfin/jellyfin-webos#31 (comment)