Skip to content
This repository has been archived by the owner on Jul 6, 2020. It is now read-only.

Recently played videos #28

Closed
frankhale opened this issue Jun 17, 2015 · 1 comment
Closed

Recently played videos #28

frankhale opened this issue Jun 17, 2015 · 1 comment
Labels

Comments

@frankhale
Copy link
Owner

Currently the top 25 videos are taken for the recently played but the algorithm is not correct. The latest video need to be the first videos on the list rather than the last. This is currently screwing up the recently played list if you have the max number of videos already.

The last paragraph doesn't make a whole lot of sense. Videos are placed on the end of the recently played list. If the video is placed after the top 25 then it will be dropped. Videos that were just played need to be put on the top of the recently played list rather than the bottom.

@frankhale
Copy link
Owner Author

I was a bit mistaken in my original diagnosis of this issue. Videos are placed at the end using push and I was taking the first 25 from the beginning of the array. I've changed that to take the first 25 from the end of the array.

@frankhale frankhale reopened this Jun 17, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant