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.

Sign up for GitHub

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

Jump to bottom

1266 private vods are still publicly listed #1340

Merged
merged 6 commits into from
Apr 23, 2024
Merged

1266 private vods are still publicly listed #1340

merged 6 commits into from
Apr 23, 2024

Commits on Mar 24, 2024

  1. Don't show private live stream to non course-admins; show live stream… 

    … of hidden courses to course admins
    @YiranDuan721
    YiranDuan721 committed Mar 24, 2024
    Configuration menu
    Copy the full SHA
    ab70631 View commit details
    Browse the repository at this point in the history

  2. Don't show private VoDs in the playlist to non course admins

    @YiranDuan721
    YiranDuan721 committed Mar 24, 2024
    Configuration menu
    Copy the full SHA
    fa96c9d View commit details
    Browse the repository at this point in the history

  3. Don't show private VoDs on the course page to non course admins

    @YiranDuan721
    YiranDuan721 committed Mar 24, 2024
    Configuration menu
    Copy the full SHA
    ed5f5d1 View commit details
    Browse the repository at this point in the history

  4. On the home page, don't show private VoDs as "LastRecording" / "most … 

    …recent VoDs" to non course admins
    @YiranDuan721
    YiranDuan721 committed Mar 24, 2024
    Configuration menu
    Copy the full SHA
    a23bd27 View commit details
    Browse the repository at this point in the history

  5. Change usage of Course.ToDTO, with a mocked admin as the parameter

    @YiranDuan721
    YiranDuan721 committed Mar 24, 2024
    Configuration menu
    Copy the full SHA
    3cc3eb2 View commit details
    Browse the repository at this point in the history

  6. No authorization in liveStreams(), as is expected in the tests. This … 

    …does not affect the result, private live streams are not listed for students.
    @YiranDuan721
    YiranDuan721 committed Mar 24, 2024
    Configuration menu
    Copy the full SHA
    87a7a21 View commit details
    Browse the repository at this point in the history