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

Watch history #182

Closed
dimqua opened this issue Sep 25, 2018 · 5 comments

Comments

@dimqua
Copy link
Contributor

commented Sep 25, 2018

Currently there is no way to see your list of watched videos, e.g. https://www.youtube.com/feed/history.

@dimqua dimqua changed the title Watch histrory Watch history Sep 25, 2018
@omarroth

This comment has been minimized.

Copy link
Owner

commented Sep 27, 2018

This is actually a bit of a limitation in how Invidious keeps watch history. As you can see here, Invidious currently stores watch history just as a list of video IDs, which is enough to filter results, but not enough to display a list of videos. A couple things will have to be changed to support this, just thought I'd provide some info.

@omarroth

This comment has been minimized.

Copy link
Owner

commented Nov 20, 2018

Added with 6b12f11. Currently, since Invidious only stores watch history as a list of video IDs, the only relevant information that can be displayed is the thumbnail. Expect this to be updated to support title and length_seconds. I'd be open to adding more information, the only concern is space.

Currently /feed/history is not linked to from anywhere. I'll probably add a link in /preferences, and maybe add it as a tab once #227 is added, although I'd appreciate thoughts on this.

@cloudrac3r

This comment has been minimized.

Copy link

commented Nov 20, 2018

I don't think the eye icon makes sense as a button to clear a video from the history — a delete icon like an x or a garbage can would make more sense.

Linking to the history in the settings sounds sensible. You could also add a link at the top of /feed/subscriptions, next to the "Manage subscriptions" link perhaps.

@dimqua

This comment has been minimized.

Copy link
Contributor Author

commented Nov 20, 2018

I'd be open to adding more information, the only concern is space.

Well, the publisher name and date would be nice to have.

@omarroth

This comment has been minimized.

Copy link
Owner

commented Mar 2, 2019

I'm going to close this since the original functionality has already been implemented.

I'm looking into refactoring to allow for the publisher name and date, as well as some other information, which should be tracked in #259, so expect to see that added soon.

@omarroth omarroth closed this Mar 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.