Fix video view all playlist#9334
Conversation
Pull request was converted to draft
Pull request was converted to draft
|
After doing some more testing I found a better way to fix this so that it uses the getChannelPlaylistId function and works for popular videos and shorts. I had to change the type for finding the continuation, but I'm not sure if that is the correct way to fix that; and it seems as though the videos load all at once rather than lazily (that's what the continuation is for, right?). I also need to figure out why the shorts only load the first 100 videos, but that will have to wait until tomorrow. |
|
Hi @r3dArch thanks for looking into this issue. There is a better way to address this one by doing this. Add play all buttons inside the relevant tabs and auto-pick the correct playlist based on the current tab and sort direction. This has the advantage of it always showing instead of only when the channel has the videos, live or shorts sections added to their home tab. Would it be possible for you to implement these changes? |
|
Hi @efb4f5ff-1298-471a-8973-3d47447115dc , I think I understand what you mean and will try to implement that soon. Would you still want the current changes as well? Thank you. |
|
Hi @r3dArch the requested changes will replace current so no need to keep current changes |
Pull Request Type
Related issue
Closes #629
Description
Creates a path for the view playlist button based on channel ID, allowing users to play all of a channel's videos.
Screenshots
Before


After
Testing
Desktop
Additional context
None