Inrixia / Floatplane-Downloader

Project for automatically organizing and downloading Floatplane videos for plex.
https://monitor.spookelton.net/public-dashboards/db0aec66747b4950b01b128916eb737e
GNU Affero General Public License v3.0
152 stars 41 forks source link

Add preference to download only captioned/uncaptioned videos #187

Closed charliedavison closed 6 months ago

charliedavison commented 6 months ago

I saw this had already been requested and is something I'd like to have the option for as well.

Not familiar at all with the FP API (or this project yet) so this might be a naive approach, but appears to work as intended.

Inrixia commented 6 months ago

This does work, though I think I'd rather implement it directly into the subchannels rather than have it as a seperate floatplane level setting since it's scoped to a specific channel rather than a site wide feature.

For now I'll leave this open though

charliedavison commented 6 months ago

@Inrixia I see, yeah that makes sense.

You're suggesting something like

{
  "title": "FP Exclusives",
  "skip": false,
  "captionVariants": {
    captioned: true,
    uncaptioned: true
  }
  "identifiers": [
     {
    "type": "channelId",
    "check": "6413623f5b12cca228a28e78"
     }
  ]
}, 

?

Or leaving the setting where it is and moving the check to the specific subchannel as its going through them?

Inrixia commented 6 months ago

I was thinking having the ability to enforce and/or in the subchannel identifiers and adding a titleStartsWith matcher. Since the captions functionality is basically just sub channel matching.

Currently there is no way to enforce and/or pairings in the identifiers though so I need to think of the best way to approach it.

I'm tempted to replace the identifiers with a string that just gets interpreted into code. Probably the most flexible.

Inrixia commented 6 months ago

@charliedavison I'm closing this as its been implemented in https://github.com/Inrixia/Floatplane-Downloader/commit/dd92e3fddf7856bb1c058d79ae307468160dbc80. That commit will likely require resetting the subscriptions in your settings as the schema has changed.

Ill update the wiki to reflect the new changes when I get the time

Inrixia commented 6 months ago

If you run into issues with the changes please open a new issue. Feel free to continue this conversation here or message me on discord. Thanks for the help