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

feat: use last_video_released for notifications #501

Closed
wants to merge 1 commit into from

Conversation

elpiel
Copy link
Member

@elpiel elpiel commented Aug 14, 2023

  • player - update on switch_to_next_video
  • notifs - update on dismiss and pull

TODO:

  • player - mark video as watched
  • player - mark as watched

- player - update on switch_to_next_video
- notifs - update on dismiss and pull

Signed-off-by: Lachezar Lechev <[email protected]>
@elpiel elpiel changed the title feat: use last_video_released for notifications: feat: use last_video_released for notifications Aug 14, 2023
@elpiel elpiel closed this Aug 21, 2023
@elpiel
Copy link
Member Author

elpiel commented Aug 21, 2023

We won't be using this field but we will keep the logic with last_watched

@elpiel elpiel deleted the feat/notifs-use-last-video-released branch January 24, 2024 09:41
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.

1 participant