Closed SimonEntholzer closed 1 day ago
The recent changes in the exam-result-overview.component.html
add conditional rendering to show grades before and after applying a bonus. If a finalGrade
exists in studentExamWithGrade.studentResult.gradeWithBonus
, the final grade is displayed, addressing the issue where the final grade was previously not shown in the exam report.
File(s) | Change Summary |
---|---|
.../exam-result-overview.component.html |
Added conditional logic to display final grade information if finalGrade is present. |
Objective | Addressed | Explanation |
---|---|---|
Display the final grade in the exam report if it exists (#8909) | ✅ |
src/main/webapp/app/exam/participate/summary/result-overview/exam-result-overview.component.html (1)
Pattern `src/main/webapp/**/*.html`: @if and @for are new and valid Angular syntax replacing *ngIf and *ngFor. They should always be used over the old style.
src/main/webapp/app/exam/participate/summary/result-overview/exam-result-overview.component.html (3)
Line range hint `1-150`: **Usage of new Angular syntax is consistent and correct.** The file consistently uses the new `@if` and `@for` syntaxes, which is in line with the latest Angular recommendations and the additional instructions provided. [APROVED] --- Line range hint `1-150`: **HTML structure and semantics are well-implemented.** The use of semantic HTML elements and accessible attributes (like `scope` for table headers) enhances the accessibility and readability of the content. --- Line range hint `1-150`: **Translations and dynamic data bindings are correctly implemented.** The extensive use of translation keys and dynamic data bindings ensures that the content is localized and responsive to the data model changes.
Checklist
General
Client
Motivation and Context
Students should be able to see their final grade when viewing the exam results.
Description
Fixes https://github.com/ls1intum/Artemis/issues/8909
Steps for Testing
Fixes https://github.com/ls1intum/Artemis/issues/8909
Testserver States
Review Progress
Performance Review
Code Review
Exam Mode Test
Test Coverage
Screenshots
Before:![image](https://github.com/ls1intum/Artemis/assets/33342534/55974b09-b963-4c00-b163-c85194c55967)
With no grade bonus:![image](https://github.com/ls1intum/Artemis/assets/33342534/5b691539-469f-4bd2-969b-431f9e05dc3c)
With grade bonus enabled:![image](https://github.com/ls1intum/Artemis/assets/33342534/b3cd487a-9563-47ed-a87f-50fcc4e4d6e8)
Summary by CodeRabbit