Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delisting unavailable pinned courses #1420

Open
wants to merge 21 commits into
base: dev
Choose a base branch
from
Open

Delisting unavailable pinned courses #1420

wants to merge 21 commits into from

Conversation

KemalKrKX
Copy link
Contributor

Motivation and Context

(#1351)

Description

The code preloads the students when preloading the course, so that if a student is not enrolled at said course/doesn't have the view right the pinned lecture is not displayed anymore.

Steps for Testing

Prerequisites:

  • 1 Lecturer
  • 1 Students
  • 1 Lecture
  1. Create a course
  2. Enroll a student to said course
  3. Pin the course as the student
  4. Set the course as enrolled only and remove the student from database
  5. Student should not see the course pinned anymore

@KemalKrKX KemalKrKX requested a review from a team December 3, 2024 16:35
Copy link

github-actions bot commented Dec 3, 2024

Your Testserver will be ready at https://1420.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

cmd/tumlive/tumlive.go Outdated Show resolved Hide resolved
docker-compose.yml Outdated Show resolved Hide resolved
.idea/dataSources.xml Outdated Show resolved Hide resolved
@karjo24 karjo24 assigned karjo24 and KemalKrKX and unassigned karjo24 Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants