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.

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

General: Migrate exercise group module to signals, inject and standalone #9891

Closed
wants to merge 36 commits into from
Closed
Changes from 1 commit
Commits
Show all changes
36 commits
Select commit Hold shift + click to select a range
cbaaf2d
migrate exercise-group.service
coolchock Nov 27, 2024
e13c7a3
migrate exercise-group-update.component
coolchock Nov 27, 2024
e4fac27
migrate exercise-groups.component
coolchock Nov 27, 2024
e582529
migrate quiz-exercise-group-cell.component
coolchock Nov 27, 2024
cea2cf3
migrate modeling-exercise-group-cell.component
coolchock Nov 27, 2024
4cd8cea
migrate file-exercise-group-cell.component
coolchock Nov 27, 2024
361b4cc
fix file-upload-exercise-group-cell.component
coolchock Nov 27, 2024
159069d
migrate programming-exercise-group-cell.component
coolchock Nov 27, 2024
342e800
fix file-upload-exercise-group-cell.component.spec.ts
coolchock Nov 28, 2024
255172a
fix modeling-exercise-group-cell.component.spec.ts
coolchock Nov 28, 2024
ee5c569
rename variable for file-upload
coolchock Nov 28, 2024
8322b8b
fix quiz-exercise-group-cell.component.spec.ts
coolchock Nov 28, 2024
65b0e6b
fix programming-exercise-group-cell.component.spec.ts
coolchock Nov 28, 2024
e9a561c
Merge branch 'develop' into chore/exercise-groups-client-migration
coolchock Nov 30, 2024
f6a99b6
Merge branch 'develop' into chore/exercise-groups-client-migration
coolchock Dec 3, 2024
662d5db
standalone imports
coolchock Dec 3, 2024
2333679
fix exercise-group-update.component.ts
coolchock Dec 3, 2024
2ba092f
fix cell tests
coolchock Dec 3, 2024
144b991
fix exercise-group-update.component.spec.ts
coolchock Dec 3, 2024
9fb875e
Merge remote-tracking branch 'origin/develop' into chore/exercise-gro…
coolchock Dec 10, 2024
31b3233
fix exercise groups tests
coolchock Dec 10, 2024
8f5ead4
fix test
coolchock Dec 12, 2024
00e3db1
export shared components
coolchock Dec 20, 2024
2c23cbd
Merge branch 'develop' into chore/exercise-groups-client-migration
coolchock Jan 2, 2025
5d48e0c
revert changes to test runs module
coolchock Jan 2, 2025
9d5f0a8
fix tests
coolchock Jan 2, 2025
0dc158e
Merge branch 'develop' into chore/exercise-groups-client-migration
coolchock Jan 2, 2025
f3cf6fa
Merge branch 'develop' into chore/exercise-groups-client-migration
coolchock Jan 3, 2025
7e121d9
remove redundant export
coolchock Jan 3, 2025
0126a3a
Merge branch 'develop' into chore/exercise-groups-client-migration
coolchock Jan 24, 2025
40ec9c9
resolve conflict
coolchock Jan 24, 2025
13c4f32
Merge branch 'develop' into chore/exercise-groups-client-migration
coolchock Jan 26, 2025
f58e5f9
Merge branch 'develop' into chore/exercise-groups-client-migration
coolchock Jan 31, 2025
533bf72
Merge branch 'develop' into chore/exercise-groups-client-migration
coolchock Feb 12, 2025
d44c7ff
Merge branch 'develop' into chore/exercise-groups-client-migration
coolchock Feb 12, 2025
c99f095
Merge branch 'develop' into chore/exercise-groups-client-migration
coolchock Feb 13, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
standalone imports
coolchock committed Dec 3, 2024
commit 662d5db98973443917fb155c35ba5802da41157d
13 changes: 1 addition & 12 deletions src/main/webapp/app/exam/manage/exam-management.module.ts
Original file line number Diff line number Diff line change
@@ -5,8 +5,6 @@ import { ExamManagementComponent } from 'app/exam/manage/exam-management.compone
import { examManagementState } from 'app/exam/manage/exam-management.route';
import { ExamUpdateComponent } from 'app/exam/manage/exams/exam-update.component';
import { ExamDetailComponent } from 'app/exam/manage/exams/exam-detail.component';
import { ExerciseGroupsComponent } from 'app/exam/manage/exercise-groups/exercise-groups.component';
import { ExerciseGroupUpdateComponent } from 'app/exam/manage/exercise-groups/exercise-group-update.component';
import { ExamStudentsComponent } from 'app/exam/manage/students/exam-students.component';
import { StudentExamsComponent } from 'app/exam/manage/student-exams/student-exams.component';
import { StudentExamDetailComponent } from 'app/exam/manage/student-exams/student-exam-detail.component';
@@ -34,10 +32,6 @@ import { ArtemisMarkdownModule } from 'app/shared/markdown.module';
import { ExamChecklistComponent } from 'app/exam/manage/exams/exam-checklist-component/exam-checklist.component';
import { ExamChecklistExerciseGroupTableComponent } from 'app/exam/manage/exams/exam-checklist-component/exam-checklist-exercisegroup-table/exam-checklist-exercisegroup-table.component';
import { ArtemisTutorParticipationGraphModule } from 'app/shared/dashboards/tutor-participation-graph/tutor-participation-graph.module';
import { ProgrammingExerciseGroupCellComponent } from './exercise-groups/programming-exercise-cell/programming-exercise-group-cell.component';
import { FileUploadExerciseGroupCellComponent } from './exercise-groups/file-upload-exercise-cell/file-upload-exercise-group-cell.component';
import { ModelingExerciseGroupCellComponent } from './exercise-groups/modeling-exercise-cell/modeling-exercise-group-cell.component';
import { QuizExerciseGroupCellComponent } from './exercise-groups/quiz-exercise-cell/quiz-exercise-group-cell.component';
import { ArtemisTextSubmissionAssessmentModule } from 'app/exercises/text/assess/text-submission-assessment.module';
import { StudentExamDetailTableRowComponent } from 'app/exam/manage/student-exams/student-exam-detail-table-row/student-exam-detail-table-row.component';
import { ExampleSubmissionsModule } from 'app/exercises/shared/example-submission/example-submissions.module';
@@ -118,8 +112,6 @@ const ENTITY_STATES = [...examManagementState];
ExamManagementComponent,
ExamUpdateComponent,
ExamDetailComponent,
ExerciseGroupsComponent,
ExerciseGroupUpdateComponent,
ExamExerciseRowButtonsComponent,
ExamStudentsComponent,
ExamStudentsAttendanceCheckComponent,
@@ -133,10 +125,6 @@ const ENTITY_STATES = [...examManagementState];
ExamChecklistComponent,
ExamChecklistExerciseGroupTableComponent,
ExamStatusComponent,
ProgrammingExerciseGroupCellComponent,
FileUploadExerciseGroupCellComponent,
ModelingExerciseGroupCellComponent,
QuizExerciseGroupCellComponent,
StudentExamDetailTableRowComponent,
ExamImportComponent,
ExamExerciseImportComponent,
@@ -152,5 +140,6 @@ const ENTITY_STATES = [...examManagementState];
StudentExamTimelineComponent,
ProgrammingExerciseExamDiffComponent,
],
exports: [ExamExerciseRowButtonsComponent],
})
export class ArtemisExamManagementModule {}
Original file line number Diff line number Diff line change
@@ -8,10 +8,14 @@ import { ExerciseGroupService } from 'app/exam/manage/exercise-groups/exercise-g
import { Exam } from 'app/entities/exam/exam.model';
import { onError } from 'app/shared/util/global.utils';
import { faBan, faSave } from '@fortawesome/free-solid-svg-icons';
import { NgbAlertsMocksModule } from '../../../../../../test/javascript/spec/helpers/mocks/directive/ngbAlertsMocks.module';
import { TranslateDirective } from 'app/shared/language/translate.directive';

@Component({
selector: 'jhi-exercise-group-update',
templateUrl: './exercise-group-update.component.html',
standalone: true,
imports: [NgbAlertsMocksModule, TranslateDirective],
})
export class ExerciseGroupUpdateComponent implements OnInit {
private route = inject(ActivatedRoute);
Original file line number Diff line number Diff line change
@@ -33,11 +33,30 @@ import { ExamImportComponent } from 'app/exam/manage/exams/exam-import/exam-impo
import { ExerciseImportWrapperComponent } from 'app/exercises/shared/import/exercise-import-wrapper/exercise-import-wrapper.component';
import { ProfileService } from 'app/shared/layouts/profiles/profile.service';
import { PROFILE_LOCALCI, PROFILE_LOCALVC } from 'app/app.constants';
import { ArtemisTranslatePipe } from 'app/shared/pipes/artemis-translate.pipe';
import { ArtemisSharedModule } from 'app/shared/shared.module';
import { ArtemisSharedComponentModule } from 'app/shared/components/shared-component.module';
import { ProgrammingExerciseGroupCellComponent } from 'app/exam/manage/exercise-groups/programming-exercise-cell/programming-exercise-group-cell.component';
import { QuizExerciseGroupCellComponent } from 'app/exam/manage/exercise-groups/quiz-exercise-cell/quiz-exercise-group-cell.component';
import { ModelingExerciseGroupCellComponent } from 'app/exam/manage/exercise-groups/modeling-exercise-cell/modeling-exercise-group-cell.component';
import { FileUploadExerciseGroupCellComponent } from 'app/exam/manage/exercise-groups/file-upload-exercise-cell/file-upload-exercise-group-cell.component';
import { ArtemisExamManagementModule } from 'app/exam/manage/exam-management.module';

@Component({
selector: 'jhi-exercise-groups',
templateUrl: './exercise-groups.component.html',
styleUrls: ['./exercise-groups.component.scss'],
standalone: true,
imports: [
ArtemisTranslatePipe,
ArtemisSharedModule,
ArtemisSharedComponentModule,
ProgrammingExerciseGroupCellComponent,
QuizExerciseGroupCellComponent,
ModelingExerciseGroupCellComponent,
FileUploadExerciseGroupCellComponent,
ArtemisExamManagementModule,
],
})
export class ExerciseGroupsComponent implements OnInit {
private route = inject(ActivatedRoute);
Original file line number Diff line number Diff line change
@@ -6,6 +6,7 @@ import { FileUploadExercise } from 'app/entities/file-upload-exercise.model';
selector: 'jhi-file-upload-exercise-group-cell',
templateUrl: './file-upload-exercise-group-cell.component.html',
styles: [':host{display: contents}'],
standalone: true,
})
export class FileUploadExerciseGroupCellComponent {
exerciseType = ExerciseType;
Original file line number Diff line number Diff line change
@@ -1,11 +1,14 @@
import { Component, input } from '@angular/core';
import { ExerciseType } from 'app/entities/exercise.model';
import { ModelingExercise } from 'app/entities/modeling-exercise.model';
import { ArtemisTranslatePipe } from 'app/shared/pipes/artemis-translate.pipe';

@Component({
selector: 'jhi-modeling-exercise-group-cell',
templateUrl: './modeling-exercise-group-cell.component.html',
styles: [':host{display: contents}'],
standalone: true,
imports: [ArtemisTranslatePipe],
})
export class ModelingExerciseGroupCellComponent {
exerciseType = ExerciseType;
Original file line number Diff line number Diff line change
@@ -9,11 +9,17 @@ import { downloadZipFileFromResponse } from 'app/shared/util/download.util';
import { AlertService } from 'app/core/util/alert.service';
import { faDownload } from '@fortawesome/free-solid-svg-icons';
import { PROFILE_LOCALVC, PROFILE_THEIA } from 'app/app.constants';
import { TranslateDirective } from 'app/shared/language/translate.directive';
import { ArtemisProgrammingExerciseStatusModule } from 'app/exercises/programming/manage/status/programming-exercise-status.module';
import { RouterLink } from '@angular/router';
import { FaIconComponent } from '@fortawesome/angular-fontawesome';

@Component({
selector: 'jhi-programming-exercise-group-cell',
templateUrl: './programming-exercise-group-cell.component.html',
styles: [':host{display: contents}'],
standalone: true,
imports: [FaIconComponent, RouterLink, TranslateDirective, ArtemisProgrammingExerciseStatusModule],
})
export class ProgrammingExerciseGroupCellComponent implements OnInit {
private profileService = inject(ProfileService);
Original file line number Diff line number Diff line change
@@ -6,6 +6,7 @@ import { QuizExercise } from 'app/entities/quiz/quiz-exercise.model';
selector: 'jhi-quiz-exercise-group-cell',
templateUrl: './quiz-exercise-group-cell.component.html',
styles: [':host{display: contents}'],
standalone: true,
})
export class QuizExerciseGroupCellComponent {
exerciseType = ExerciseType;