Lecture: Allow instructors to remove solution slides when processing units

[pellumbbaboci]

Checklist

General

• I tested all changes and their related features with all corresponding user types on a test server.
• 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.

Server

• Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
• I followed the coding and design guidelines.
• I added multiple integration tests (Spring) related to the features (with a high test coverage).
• I documented the Java code using JavaDoc style.

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.
• Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
• I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
• I documented the TypeScript code using JSDoc style.
• I added multiple screenshots/screencasts of my UI changes.
• I translated all newly inserted strings into English and German.

Motivation and Context

Instructors are not able to remove "Example solution" slides when processing units automatically.

Description

A new checkbox is provided to choose if the instructors want to remove the solution slides. Then all example solution slides that contain "Example solution", "Example solution: model" or "Example solution (with comments)" will be removed.

Steps for Testing

Prerequisites:

• 1 Instructor
• 1 Course

1. Log in to Artemis
2. Navigate to Course Administration
3. Create a new Lecture and choose to automatically process the file.
4. Then you will see a table with split information and at the bottom you will have the option to choose to remove Break or Solution slides.
5. Select both and then submit the form.
6. See the file units if the slides are removed correctly.
   example pdf:

ITP2223 L05 Object Orientation I_organized.pdf

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
• I confirm that the server changes (in particular related to database calls) 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)
attachment-units.component.ts	89.47%	✅

Server

Class/File	Line Coverage	Confirmation (assert/expect)
LectureUnitProcessingService.java	97%	✅
LectureUnitInformationDTO.java	100%	✅

�

Screenshots

New checkbox introduced:

[Strohgelaender]

This PR only removes the sildes, it does not allow the instructor to reveal them again after the lecture ended. Can you add this as well to the PR?

[pellumbbaboci]

This PR only removes the sildes, it does not allow the instructor to reveal them again after the lecture ended. Can you add this as well to the PR?

@Strohgelaender no, thats what its intended. The reverting back process should be addressed in another PR as it requires bigger changes.

[nityanandaz]

I confirm that example solutions slides are removed (ts1)

[DominikRemo]

Tested in testing session on ts1. Works as described

However, I'd love to see @JohannesStoehr`s suggestion implemented in a follow up :)

Why not offer the user a text box next to the option of removing the slides?
This way the user can choose the text themselves and it works for all lectures in all languages

[lennart-keller]

Code looks fine :)

[JohannesStoehr]

Code looks good to me.
Please keep the Follow up for user defined strings in mind