whyboris / Video-Hub-App

Official repository for Video Hub App
https://videohubapp.com
MIT License
568 stars 174 forks source link

Add an option to create a non-recursive hub. #818

Open psultan opened 1 year ago

psultan commented 1 year ago

It would be nice to have an option to set a specific directory as a hub without recursively including videos in subdirectories.

Yes, it is possible to use filters to narrow down the results to only a single folder but that is a much more obtuse workflow that also requires significantly more, unnecessary data processing.

whyboris commented 1 year ago

Thank you for a suggestion for Video Hub App 🙌

I'm assuming you are recommending this happens during the creation of a hub (or in a follow-up step of adding another directory into VHA). It does seem like it's very easy to create a folder manually (in your OS), drag all the videos you want into it, and then import just that folder into VHA.

Is there any reason that's not a good-enough solution?

psultan commented 1 year ago

Good question, ideally it would be more flexible/customizable if the setting could be applied for each folder in a hub, however, if that is significantly more work, setting it per hub is probably a workable compromise. I think I would be using it non-recursive 99% of the time.

Yes, theoretically that is a potential solution. However, from a user/datahoarder perspective we have already spent significant time organizing our media library. Unfortunately, breaking that organization nullifies the benefits of using this tool.

whyboris commented 1 year ago

Since I have the SourceFolder interface (an object rather than a string) - I believe I can throw in an extra boolean in there which says "recursive" or not 🤔

https://github.com/whyboris/Video-Hub-App/blob/main/interfaces/final-object.interface.ts#L10

But the UI (user interface) feels a bit more complicated. We'd need a checkbox next to the source folder when selecting it (during hub creation), and then somewhere in the Current hub tab (4th tab) in settings - next to each of the folders on the "Video folders located here" ... and perhaps yet another Add folder to hub, nonrecursively button to the already-cluttered 4 buttons I have (after you click edit folders) 😅

Visually that is already a bit cluttered, but it may be possible with a bunch of work 🤔

I still have a hard time wrapping my head around the use case. If you have, for example, a folder /movies which has many sub-folders, and you only want to add /movies/comedies but for some reason you don't want to include the sub-folder /movies/comedies/international 🤯

I have been asked to allow to exclude a folder (which I understand -- maybe your /movies has one folder that says /movies/incomplete-downloads or something 🤔 ) ...

I'll be honest I'm still a bit reluctant to add the feature you are requesting -- but perhaps you could share more of your motivation?

Full disclosure -- I finally have found time going forward to resume work on VHA and my first order of business is to try to get the (optional!) facial recognition (all local on each person's machine, never going on the internet). But when I'm done releasing that (maybe winter 2023) I'll be on the lookout for any other features ❤️