ConfusedPolarBear / intro-skipper

Fingerprint audio to automatically detect and skip intro sequences in Jellyfin
GNU General Public License v3.0
988 stars 392 forks source link

Update Jellyfin-web to 10.8.9 #128

Closed JVT038 closed 1 year ago

JVT038 commented 1 year ago

The current version is 10.8.5 It would be nice to make use of 10.8.9, as it contains the latest bug fixes and such.

Mavyre commented 1 year ago

Tried on 10.8.8 and that works for me :)

aleksasiriski commented 1 year ago

I'll make a docker image which uses official latest jellyfin and changes to this webui with intro skipper today, so watch my repos if you want

Anirte commented 1 year ago

I'll make a docker image which uses official latest jellyfin and changes to this webui with intro skipper today, so watch my repos if you want

Well, if not OP, I will wait, cause official docker image doesn't work for me

aleksasiriski commented 1 year ago

I'll make a docker image which uses official latest jellyfin and changes to this webui with intro skipper today, so watch my repos if you want

Well, if not OP, I will wait, cause official docker image doesn't work for me

I was a bit busy yesterday sry, making it now with and without rffmpeg (I used to have one but deleted it after remaking my docker images)

aleksasiriski commented 1 year ago

https://github.com/aleksasiriski/jellyfin-rffmpeg-intro-skipper

Mavyre commented 1 year ago

@aleksasiriski Pretty cool. However, that updates the server to 10.8.9 while jellyfin-web stays at the latest commit made by the PolarBear, which is 10.8.5. I think we'd have a little work to merge the 10.8.9 from jellyfin-web to fully update it.

Honestly, I can't wait for a script injection to be made for that plugin 😄

aleksasiriski commented 1 year ago

I can try to add building steps of latest jellyfin-web from this repo but that will only make sense if @ConfusedPolarBear updated the web from upstream. If he didn't I don't want to mess with that

roib20 commented 1 year ago

I see the confusedpolarbear/jellyfin-intro-skipper Dockerfile was updated today to use jellyfin/jellyfin:10.8.9 as the base image. However on ghcr.io/confusedpolarbear/jellyfin-intro-skipper the latest tag is still for 10.8.5 rather than 10.8.9.

Mavyre commented 1 year ago

My bad. @ConfusedPolarBear just updated both repos. Thanks BTW!

roib20 commented 1 year ago

I made Pull Request #132 to help improve the image functionality and add multi-arch support (AMD64, ARM and ARM64). If anyone wants to help test, the images are available over on my fork: docker pull ghcr.io/roib20/jellyfin-intro-skipper:10.8.9 or latest. Pi testers for the ARM images would be very helpful.

roib20 commented 1 year ago

@aleksasiriski Pretty cool. However, that updates the server to 10.8.9 while jellyfin-web stays at the latest commit made by the PolarBear, which is 10.8.5. I think we'd have a little work to merge the 10.8.9 from jellyfin-web to fully update it.

Honestly, I can't wait for a script injection to be made for that plugin smile

After further checking it appears you are correct. ConfusedPolarBear / jellyfin-web is not yet fully updated. This is not to be confused with the server back-end which is updated to v10.8.9.

ConfusedPolarBear commented 1 year ago

The current version is 10.8.5

It would be nice to make use of 10.8.9

Closing, as both the container (Dockerfile, image) and the web interface fork have been updated to 10.8.9.