Skip to content
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

Continue watching #236

Draft
wants to merge 54 commits into
base: main
Choose a base branch
from
Draft

Continue watching #236

wants to merge 54 commits into from

Conversation

iBicha
Copy link
Owner

@iBicha iBicha commented Dec 21, 2023

Closes #138 and #234

  • Add router that returns videos for continue watching, including watch date and progress (returns them sorted and paginated)
    • It would return small page size (like 5) but includes video info?
  • Add a node for storing videos in memory, and write to registry, with a limit of 100 videos
    • Should contain a subnode for each video, so that renderers can observe changes
  • Add settings
    • Enable/disable continue watching
    • Max days to keep videos (videos expire based on last watch date)
    • Video count (capped at 100)
    • A button to clear all videos
    • Web app controls
  • Videos with about 95% progress (capped at 20 seconds) should be considered watched
  • Add progress bar to video cell (this means "progressNode" is going to be a field in VideoContentNode)
    • Also add it to PlaylistViewCell
  • Add support for web app
    • Rendering the new feed
    • Casting / Queuing the video should continue from the last watch position
  • Settings
    • Enable/disable continue watching should disable the feed, and prevent adding new videos
    • Max days to keep videos should be checked on load, and remove old videos
    • Video count (capped at 100) should be checked on adding new videos, and remove old videos
  • Handle case where existing VideoContentNode is not aware of VideoProgressContentNode created later
  • Continue watching feed need to be reactive to change in the continue watching list
  • Introduce concept of hidden feed if empty
    • Roku
    • Web app
  • Update openapi spec file
  • [Tentative] context menu to delete item from continue watching feed

iBicha and others added 30 commits December 20, 2023 16:21
# Conflicts:
#	playlet-lib/src/components/Services/Invidious/Invidious.bs
#	playlet-lib/src/components/VideoPlayer/VideoPlayer.bs
# Conflicts:
#	playlet-lib/src/components/PlayQueue/PlayQueue.bs
#	playlet-lib/src/components/PlaylistView/PlaylistView.bs
#	playlet-lib/src/components/Screens/SettingsScreen/SettingsScreen.bs
#	playlet-lib/src/components/VideoFeed/VideoRowList.bs
#	playlet-lib/src/components/VideoPlayer/VideoPlayer.bs
github-action linter and others added 24 commits December 25, 2023 01:21
# Conflicts:
#	playlet-lib/src/components/VideoFeed/VideoRowCell/VideoRowCell.bs
# Conflicts:
#	playlet-lib/src/components/VideoPlayer/VideoPlayer.bs
# Conflicts:
#	playlet-lib/src/components/Services/Preferences/Preferences.bs
#	playlet-web/src/lib/Screens/Settings/SettingsNode.svelte
#	playlet-web/src/lib/Stores.ts
# Conflicts:
#	playlet-lib/src/components/VideoPlayer/VideoPlayer.bs
# Conflicts:
#	playlet-lib/src/components/VideoPlayer/VideoPlayer.bs
# Conflicts:
#	playlet-lib/src/components/Web/PlayletWebServer/Middleware/PlayQueueRouter.bs
#	playlet-lib/src/components/Web/PlayletWebServer/Middleware/ViewRouter.bs
#	playlet-lib/src/components/Web/PlayletWebServer/PlayletWebServer.bs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remembering Playback Position and Watch History
1 participant