Closed Jubair70 closed 1 year ago
thanks for the pull request brother. I would like to request you to allow me some time (2 weeks) to complete my current refactoring on the server side (making genric repo thing). then i can accept this PR. Thanks 🙏
@Jubair70 please have a look https://github.com/foyzulkarim/mern-video-streaming/issues/51
this pr has conflicts.
when you complete your changes, please use the label ready for review
this feature should contain the implementation of server side as well.
when you complete your changes, please use the label ready for review
Hello @foyzulkarim vai, I don't think I have enough permission to change the label. I have updated the PR. Could you please take a look ? Thanks!
Looks good for now. actual implementation of sorting will be handled in another ticket.
This pr will close https://github.com/foyzulkarim/mern-video-streaming/issues/30
sort by view count, then newer first
, I have implemented it asPopular
Popular
category sort the videos by views count first. if the view count equals then it will sort by date and time in descending order.Popular
category by default.