-
-
Notifications
You must be signed in to change notification settings - Fork 231
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
[Feature Request]: User created playlist #590
Comments
Well, this is already an offline application. TA doesn't need internet connection to work, after you have downloaded and indexed your videos obviously. Should also be portable, like move a portion of TA to you laptop temporarily, all persistent data is in the volume, or if you know you'll be offline later, sync your data first, then you can use TA offline perfectly well. Just saying that you can do that already. To the rest, obviously, please go ahead, any of the documented bug reports or feature requests are ready to be taken on by you. |
ok I think I will start on the playlist idea first (existing roadmap feature). The offline scenario is for an Android phone. I goto the gym and they don't have good WiFi, I don't have an unlimited cell data plan, but would still like to watch videos from my phone. |
Ah, I see, you mean offline from the TA server and not from the internet. Honestly that sounds like out of scope for this project, I don't see how this could be done with caching alone, as the interface constantly interacts with the backend. Maybe some sort of static site generator would be a better approach? Or some sort of export script that can just get the video files you want and makes a copy to your phone? So you can use the phones built in media player? |
Looks great! A lot of people have been asking for this feature, and will make a great addition :) |
Nice, some great progress here! Not sure how well this is going to work once you have a dozen plus playlists? That will mess up the layout there. Maybe the controls for adding a video to a playlist would be better Edit: |
take a look at how youtube does this. It is similar to this approach. I can add scrollbar to drawer menu to limit impact on layout. If it turns out the drawer is a problematic, I can switch it to a popup centered around the tile, but I imagine that will come with it's own complexity to solve. I'm trying to limit this feature to 3 days or less (24 hrs or less) so the idea here is to get some basic features working so we can take a look at it on the testing branch. |
I sent a preview pull request for this feature. I didn't see an option to associate the pull request to this ticket. Can you tell me how to do that? |
Are you still working on this? Ideally this should be tied together with #609, to make playlists extendible by type. I'd suggest to set a So |
yes. playlist_type makes sense seeing the other feature request. I chose an is_custom boolean field, but I can refactor into the playlist_type. One issue that came up when inspecting the code is how we want to handle the existing playlists. I'd like to create a routine during the upgrade process that will add this field if it does not exist. Otherwise the elastic search queries and view logic contain many conditional statements. |
For the migration, you can take inspiration from how this has been handled before in similar cases, for example when we introduced auto start in v0.3.6: Basically create a query similar to that, but if must not exist field With that approach you don't have to worry about any temporary logic within the application. And with the must not exist logic, you don't need to worry to overwrite legitimate That has been the most robust approach, at least from what I have come up with so far. Plus it's easy to remove after a few releases and we don't clutter up the code base with all these temporary migration logic. |
agree will do. this will need a good testcase |
I prefer to use Jellyfin to watch the TA videos. In there I can download the videos before going out to my phone or ask it to transcode it to a lower bitrate if I want to watch it with limited data plan. |
Thanks K1 I will have to install Jellyfin and take a look. I recall looking at their documentation and not seeing what I wanted. I like watching the youtube content from a list. So the concept I am looking for is: add/remove content from a list. Sync a device (once) with a list. Device auto downloads the list, device auto updates the list. I think this can be done with a combination of this feature, plus a real small new feature that provides a link to the 'podcast' format of the list. I think there are a lot of apps out there that understand the podcast format. Podcasts genre is update frequently (weekly basis for example), so I imagine the app developers have thought of this and have an offline solution to my use case. I'll have to see if there is a simple solution out there. |
code update complete. Pull request issued at: #620 |
requested additional permissions from admin so I can resolve merge conflict on testing branch. |
implemented and merged with v0.4.7, shutout to @greg321321 |
I've read the documentation
Your Feature Request
I would like to build a Watch Offline feature into this project.
Use Case: When at a location where there is no internet connection, I would like to view videos on my PED (Personal Electronic Device).
User Stories:
I believe all this work can be done simply using web caching. Let me know if you think this is a good idea. I can build the feature with your direction and knock out one or two of your roadmap features/bugs in the process. I am a software developer familiar with web technologies/gitlab.
Your help is needed!
The text was updated successfully, but these errors were encountered: