search

jellyfin / jellyfin-webos

WebOS Client for Jellyfin
https://jellyfin.org
Mozilla Public License 2.0
638 stars 65 forks source link

[Feature] Remember last connected servers #82

Closed vvvrrooomm closed 4 months ago

vvvrrooomm commented 2 years ago

Sever auto-discovery introduced a nice way to switch between local servers. In a setup with (at least) one remote server which is not auto-discoverable we have to type the server url again after every switch.

It would be nice to remember the last (LRU) connected servers and add them to the list of auto-discovered items. Remembered servers are displayed first, auto-discovered next. Servers which are both, remembered and auto-discovered are displayed only once (as long as the id did not change). The last 3 servers are stored A different color could be used to distinguish the 2 types of servers.

This PR is a showcase to find out if such functionality is desired, it works on my TV (webOS5.3) and now also tested on webos3.0 emulator If acceptable testing for other webos versions can be done

anthonylavado commented 2 years ago

I apologize for the super long delay on this. I had the wrong notification settings set. As a PR with features, I like it. Since it will change the first screen flow, I won't include it on the first release, but as an update right after. I want to keep the release as similar as possible to everything tested, and LG gets you to write a user flowchart for navigating all the screens, which I wouldn't have a chance to update right now.

I will get this in soon though.

e-bacho commented 11 months ago

I just realised the PR I just opened is kind of similar to this. Would love to get an update from someone on whether we can merge this (and when).

anthonylavado commented 9 months ago

@e-bacho I'm so sorry for the extra super delayed response. I'll merge whichever works better. I've had to recreate the LG SDK setup again for the third time on my system, and want to get something in for webOS 4.0. I know it's subjective, but because it might take me a while to get sim/emu set up, let me know which PR you'd want me to look at first. This one, or yours?

e-bacho commented 9 months ago

@anthonylavado Thank you for your response. This PR is probably better suited, it'd be good to get this merged in. I am happy to close my PR.

It's an absolute pain typing in the server URL on the TV each time you change between multiple servers which should technically be remembered.

Much appreciated!