Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Emby 2.2.9 backport plus Jellification #7

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ShaheedHaque
Copy link

@ShaheedHaque ShaheedHaque commented Jul 3, 2021

  1. Get updated Jellyfin images etc from https://github.com/computersandstuff/jellyfin-samsungtv-updated.
  2. Backport Emby2.2.9 application code.
  3. Add installation notes and a helper script.

computersandstuff and others added 4 commits December 27, 2020 18:55
The splash screen, in app logos, and launch icon logos have been changed to Jellyfin's logo which was taken directly from the img folder from Jellyfin web. The user accounts icon and rotten tomatos ratings have been updated as well as the player controls and slider being changed to the Jellyfin purple insdead of the emby green. The name of the app was changed to Jellyfin in various locations except for the credit page as I thought it better to leave the names of the original contributers and emby's website.

No major code was changed and I understand that this is an old app for older samsung TVs (like my own), but I as I am still learning C# I wanted to start contributing in a much smaller area like this one.
@dkanada
Copy link
Member

dkanada commented Jul 4, 2021

Do you have an older Samsung TV or something? Most of our Samsung work is on the new Tizen repo and we don't plan on supporting this client in the future.

@ShaheedHaque
Copy link
Author

Yes, I do. I'm not overly concerned that you guys should spend time on this, at least part of my thinking is that anyone else that cares will see this PR and may instead just pull the code from my fork.

But if it is merged, then I may continue to play around with the code... On a similar basis...

@dkanada
Copy link
Member

dkanada commented Jul 4, 2021

Since it's supposedly impossible to release new apps on Orsay TVs we probably wont be merging anything here. If you did want to maintain a client you're welcome to add it to our docs in the list of available clients though.

@ShaheedHaque
Copy link
Author

Since it's supposedly impossible to release new apps on Orsay TVs we probably wont be merging anything here. If you did want to maintain a client you're welcome to add it to our docs in the list of available clients though.

IIUC this repo is for users who want to install the app via a local web server (or maybe USB) since the Samsung "app store" is not longer hosting new content. So I'm not sure I understand why that might influence the decision to merge?

[Though as I said before, I'm OK with not merging if that is deemed the right way forward].

@josecarre
Copy link

Thanks for your effort @ShaheedHaque , i'm using your fork and works fine :), lets save the world reusing our old tvs till they die!

@joao-milheirico
Copy link

joao-milheirico commented Apr 26, 2024

@ShaheedHaque doing the lords work here 😄 thanks, working 2.2.9 on a ES5500 from 2012

@oherau
Copy link

oherau commented Oct 30, 2024

I recently had network issue with the latests versions of the Jellyfin server (10.10+)
I tried several tweaks on the server side but nothing worked.
I rembered the old client was a fork of Emby so I seach in Samsung Orsay Official (old) store, and I was lucky enough and rather surprised to find a legit old version of the gueniune Emby client.

I installed it and guess what? it works out of the box !
Cherry on the cake: some features that didn't work on the retro-port of the jellyfin client works with the official Emby client.

So this is my advice: on old Samsung Orsay models, just use Emby client if available on the official store.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

6 participants