Add setting for shuffling subscription feed#8974
Conversation
Added a setting which allows the option to randomize the order of videos in the subscription feed
github-actions
Bot
added
the
PR: waiting for review
ChunkyProgrammer
changed the title
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
github-actions
Bot
removed
the
PR: waiting for review
Head branch was pushed to by a user without write access
|
Conflicts have been resolved. A maintainer will review the pull request shortly. |
absidue
left a comment
absidue
left a comment
There was a problem hiding this comment.
Looks like the merge wasn't clean, please correct that. Additionally the shuffling should be done in the SubscriptionTab component, please also look at the more efficient shuffling implementation that the WatchVideoPlaylist component is using, which avoids creating unnecessary extra objects and arrays.
|
This PR is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 14 days. |
|
This PR was closed because it has been stalled for 14 days with no activity. |
3 participants
Added a setting which allows the option to randomize the order of videos in the subscription feed
Pull Request Type
Related issue
closes #7907
Description
Adds a "shuffle" mode to the subscription video list
This takes the videos which would normally be shown in the subscription list and then displays them in a random order
Testing
First go to subscriptions page and observe the videos are in order by upload date descending
Go to the settings page and apply the "Randomize Order" toggle
Go back to subscriptions page and observe the same videos are still displayed however in a random order
Desktop