search

TUM-Dev / gocast

TUMs lecture streaming service.
https://tum.live
MIT License
186 stars 42 forks source link

1185-recent-VODs-section-doesnt-show-pinned-public-courses #1221

Closed barisgul15 closed 11 months ago

barisgul15 commented 11 months ago

Motivation and Context

Issue 1185: Recent VODs section doesnt show pinned (public) courses

Description

Before the changes, only user courses were listed for the recently VOD section. Now there is a new array for pinned Courses and if it is detected that user has one or more pinned courses, pinned and user courses are merged into a new array (with eliminating duplicates) and this array is used for the recently VOD section. However a user without any course will still be seeing all public courses in recently VOD section whether he/she has pinned any course or not. That can also be changed to see only pinned courses if desired, but since the probability of a user without any course pinning a course is so low, that feature is not implemented.

Steps for Testing

  1. Log in
  2. Pin a public course that is not also listed in my courses.
  3. Navigate to recently VOD section.

Screenshots

github-actions[bot] commented 11 months ago

Your Testserver will be ready at https://1221.test.live.mm.rbg.tum.de in a few minutes.

Logins | Kurs1 | Kurs2 | Kurs3 | Kurs4 | | ---------------------------------------- | ------------------------- | ------------------------- | ------------------------- | | public | public | loggedin | enrolled | | prof1 | prof1 | prof2 | prof1
prof2 | | student1
student2
student3 | student1
student2 | student2
student3 | student1
student2 |