Development: Remove Course Management and Admin modules

[tobias-lippert]

Checklist

General

• I tested all changes and their related features with all corresponding user types 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 (e.g. using paging).
• I strictly followed the client coding and design guidelines.
• I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
• I documented the TypeScript code using JSDoc style.

Motivation and Context

We want to remove all ngModules step by step to completely follow modern Angular best practices without modules and standalone components instead. This ensures the long-term maintainability of the Artemis client application.

Description

Removed all modules related to course management and LTI and adapted routes as necessary.

Steps for Testing

Prerequisites:

• 1 Instructor
• 1 Admin

1. Log in to Artemis
2. Navigate to Course Administration
3. Test everything in the course management
4. Create programming/modeling/quiz/file upload/text exercise
5. Import programming/modeling/quiz/file upload/text exercise
6. Edit programming/modeling/quiz/file upload/text exercise
7. Inspect the course scores/statistics
8. Assess a submission
9. Try to access the complaint dashboard
10. Try to access the feedback requests dashboard
11. Create a tutorial group
12. Create a competency, edit a competency
13. Navigate to the conversations tab
14. Create/edit lecture
15. Make sure you can access all the pages linked in the top navigation bar of the course management
16. Click wherever you want and make sure no routes are broken
17. Check all admin menu points are still reachable.

Exam Mode Testing

Prerequisites:

• 1 Instructor
• 1 Students
• 1 Exam with all exercise types

1. Log in to Artemis
2. Participate in the exam as a student
3. Make sure nothing looks broken and all routes work as expected

Testserver States

Note

These badges show the state of the test servers.
Green = Currently available, Red = Currently locked
Click on the badges to get to the test servers.

Review Progress

Performance Review

• I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance even for very large courses with more than 2000 students.

Code Review

• Code Review 1
• Code Review 2

Manual Tests

• Test 1
• Test 2

Exam Mode Test

• Test 1
• Test 2

Test Coverage

changed files only affect routes and modules. For those files, coverage is not a suitable metric.

Summary by CodeRabbit

Summary by CodeRabbit

• New Features

  • Added new resolvers to enhance data loading for programming and text-based assessments.
  • Introduced new components for participant score distribution and export functionality in exam scores.
  • Enhanced the functionality of the assessment dashboard with new components for tutor participation and leaderboard.

• Refactor

  • Simplified routing configurations with streamlined URL structures by removing extraneous parameters.
  • Consolidated and removed deprecated modules to improve overall system organization and performance.
  • Updated routing logic by replacing module imports with direct component references.

• Chores

  • Updated dependency management by replacing module-based imports with targeted component imports.
  • Conducted extensive code cleanup across various features to ensure consistency and maintainability.

[HawKhiem]

Tested on TS3. Everything works as expected!

[SimonEntholzer]

reapprove

[Fallthunderclap]

Everything works as expected!

[HawKhiem]

Reapprove

[coolchock]

re-approve code after merge conflict was resolved

[zagemello]

Tested on TS3, all works as expected

[kevinfischer4]

Tested on TS5, all mentioned points work as expected 👍