PeerTube icon indicating copy to clipboard operation
PeerTube copied to clipboard

feat: show contained playlists under My videos

Open kontrollanten opened this issue 3 years ago • 3 comments

Description

Shows which playlists videos are added to under My Videos.

I couldn't figure out how the cache is working in the VideoPlaylistService, so now these calls aren't added to the cache.

Related issues

closes #4769

Has this been tested?

  • [x] 👍 yes, I added tests to the test suite

Screenshots

image

kontrollanten avatar Jul 13 '22 00:07 kontrollanten

Hello,

  • Server files should not change since you only use existing server methods
  • Miniature should check itself playlists existence, like we do for the watch later playlist

Chocobozzz avatar Jul 19 '22 08:07 Chocobozzz

  • Server files should not change since you only use existing server methods

How should we get the playlist name and short uuid then? It sounds inefficient to do extra API calls to just get this data.

  • Miniature should check itself playlists existence, like we do for the watch later playlist

Then we'll get at least a 500 ms delay which I think is a bad user experience.

kontrollanten avatar Jul 19 '22 15:07 kontrollanten

How should we get the playlist name and short uuid then? It sounds inefficient to do extra API calls to just get this data. Then we'll get at least a 500 ms delay which I think is a bad user experience.

Okay then :+1:

Chocobozzz avatar Jul 25 '22 12:07 Chocobozzz

Thanks!

Chocobozzz avatar Oct 24 '22 12:10 Chocobozzz