search

insidegui / WWDC

The unofficial WWDC app for macOS
https://wwdc.io
BSD 2-Clause "Simplified" License
8.63k stars 783 forks source link

Auto-delete watched videos #567

Closed Blackjacx closed 5 months ago

Blackjacx commented 5 years ago

It would be cool to provide an option in the settings to auto-delete watched session videos so they do not clutter up disk space.

Specs

insidegui commented 5 years ago

Removing the "Good first issue" tag since as it turns out, this is not as simple as it sounds 😅 (Developing in #570).

bcmn commented 5 years ago

Closed #570 as I'm not likely to revisit it immediately. Just to capture...

• Needs toggle in Preferences • Must respect existing "watched" definition, used for filters, etc. • Should probably sweep for watched videos on first enabling • Avoid deleting newly-watched videos until player torn down (it shouldn't be disastrous, but seems unwise) -- but we don't have a stop state for videos, which makes this a little tricky • Should probably delete immediately if "mark as watched" used

There's some interesting obstacles in the code to get around to achieve these.

Alternatively, maybe we run a sweep semi-regularly (on launch?) whenever the feature is enabled.