Lectures: Allow sorting of lectures in the course management

[basak-akan]

Checklist

General

• This is a small issue that I tested locally and was confirmed by another developer on a test server.
• Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
• I chose a title conforming to the naming conventions for pull requests.

Client

• Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data.
• I followed the coding and design guidelines.
• I added multiple screenshots/screencasts of my UI changes.

Motivation and Context

Resolves #6439
This feature would make it easier for users to find specific lectures, especially in courses with a large number of lectures. The sorting feature would enhance the user experience by providing a more organized view of lectures.

Description

A lecture sorting feature is added in the course management section of Artemis. This feature allows users to sort lectures based on different criteria such as id, title, description, visible from, start, and end date.

• Alphabetical sorting for string values adapted into existing sort service
• Course column removed

Steps for Testing

Prerequisites:

• 1 Instructor/Admin/Editor/TA (who has access to course management)
• Course which has multiple lectures

1. Log in to Artemis
2. Navigate to Course Management
3. Navigate Lectures
4. Verify sorting lectures by id, title, description, visible from, start, and end date possible.

Review Progress

Performance Review

• I confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance

Code Review

• Code Review 1
• Code Review 2

Manual Tests

• Test 1
• Test 2

Test Coverage

Client

Class/File	Line Coverage	Confirmation (assert/expect)
lecture.component.ts	96.47%	✅

Screenshots

[terlan98]

The changes look good. I have one comment regarding the new test 🙂

[github-actions]

❌ Unable to deploy to test servers ❌

Testserver "artemis-test6.artemis.cit.tum.de" is already in use by PR #6760.

[JohannesStoehr]

Code looks good to me

[terlan98]

Code

[julian-christl]

Thank you for the changes, code looks great. Also quickly tested on TS1

[julian-christl]

I suggest changing the title to Lectures. Not sure what it has to do with Assesssment.

=> "Lectures: Allow sorting in the course management"

[Strohgelaender]

code looks good

[laadvo]

Tested on ts1, lectures can get sorted by different criteria as expected. Useful feature :)

Outdated