Skip to content

Commit 2035f5e

Browse files
ypinhsuanjasonbchniqiukunrrtheonlyone
authored
[#10597] Instructor edit session page: page refreshes by itself on first loading (#11172)
* Remove editSession and editSessionEventHandle for instructor * Remove editSessionEvent in sessions-table.component.ts Co-authored-by: PHONG BACH <phongnghinhbach@gmail.com> Co-authored-by: Ni Tianzhen <nitianzhen1210@gmail.com> Co-authored-by: Rahul Rajesh <rahul.rajesh.bhat@gmail.com>
1 parent 4695b3e commit 2035f5e

File tree

8 files changed

+3
-33
lines changed

8 files changed

+3
-33
lines changed

src/web/app/components/sessions-table/__snapshots__/sessions-table.component.spec.ts.snap

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ exports[`SessionsTableComponent should snap like in home page with 2 sessions so
1212
copySessionEvent={[Function EventEmitter]}
1313
courseCandidates={[Function Array]}
1414
downloadSessionResultsEvent={[Function EventEmitter]}
15-
editSessionEvent={[Function EventEmitter]}
1615
headerColorScheme="0"
1716
loadResponseRateEvent={[Function EventEmitter]}
1817
moveSessionToRecycleBinEvent={[Function EventEmitter]}
@@ -446,7 +445,6 @@ exports[`SessionsTableComponent should snap like in sessions page with 2 session
446445
copySessionEvent={[Function EventEmitter]}
447446
courseCandidates={[Function Array]}
448447
downloadSessionResultsEvent={[Function EventEmitter]}
449-
editSessionEvent={[Function EventEmitter]}
450448
headerColorScheme="0"
451449
loadResponseRateEvent={[Function EventEmitter]}
452450
moveSessionToRecycleBinEvent={[Function EventEmitter]}
@@ -845,7 +843,6 @@ exports[`SessionsTableComponent should snap with default fields 1`] = `
845843
copySessionEvent={[Function EventEmitter]}
846844
courseCandidates={[Function Array]}
847845
downloadSessionResultsEvent={[Function EventEmitter]}
848-
editSessionEvent={[Function EventEmitter]}
849846
headerColorScheme="0"
850847
loadResponseRateEvent={[Function EventEmitter]}
851848
moveSessionToRecycleBinEvent={[Function EventEmitter]}

src/web/app/components/sessions-table/sessions-table.component.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@
6565
<ng-container *ngTemplateOutlet="editSessionBtn"></ng-container>
6666
</a>
6767
<ng-template #editSessionBtn>
68-
<button type="button" class="btn btn-light btn-sm" (click)="editSessionEvent.emit(idx)"
68+
<button type="button" class="btn btn-light btn-sm"
6969
[disabled]="!sessionsTableRowModel.instructorPrivilege.canModifySession">
7070
Edit
7171
</button>

src/web/app/components/sessions-table/sessions-table.component.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,9 +55,6 @@ export class SessionsTableComponent implements OnInit {
5555
@Output()
5656
loadResponseRateEvent: EventEmitter<number> = new EventEmitter();
5757

58-
@Output()
59-
editSessionEvent: EventEmitter<number> = new EventEmitter();
60-
6158
@Output()
6259
moveSessionToRecycleBinEvent: EventEmitter<number> = new EventEmitter();
6360

src/web/app/pages-instructor/instructor-home-page/instructor-home-page.component.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ <h1>Home</h1>
9090
[sessionsTableRowModelsSortOrder]="courseTabModel.sessionsTableRowModelsSortOrder" [courseCandidates]="courseCandidates"
9191
[headerColorScheme]="SessionsTableHeaderColorScheme.WHITE" [columnsToShow]="[ SessionsTableColumn.START_DATE, SessionsTableColumn.END_DATE ]"
9292
(sortSessionsTableRowModelsEvent)="sortSessionsTableRowModelsEvent(idx, $event)"
93-
(loadResponseRateEvent)="loadResponseRateEventHandler(idx, $event)" (editSessionEvent)="editSessionEventHandler(idx, $event)"
93+
(loadResponseRateEvent)="loadResponseRateEventHandler(idx, $event)"
9494
(moveSessionToRecycleBinEvent)="moveSessionToRecycleBinEventHandler(idx, $event)" (copySessionEvent)="copySessionEventHandler(idx, $event)"
9595
(submitSessionAsInstructorEvent)="submitSessionAsInstructorEventHandler(idx, $event)" (viewSessionResultEvent)="viewSessionResultEventHandler(idx, $event)"
9696
(publishSessionEvent)="publishSessionEventHandler(idx, $event)" (unpublishSessionEvent)="unpublishSessionEventHandler(idx, $event)"

src/web/app/pages-instructor/instructor-home-page/instructor-home-page.component.ts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -311,13 +311,6 @@ export class InstructorHomePageComponent extends InstructorSessionModalPageCompo
311311
this.loadResponseRate(this.courseTabModels[tabIndex].sessionsTableRowModels[rowIndex]);
312312
}
313313

314-
/**
315-
* Edits the feedback session.
316-
*/
317-
editSessionEventHandler(tabIndex: number, rowIndex: number): void {
318-
this.editSession(this.courseTabModels[tabIndex].sessionsTableRowModels[rowIndex]);
319-
}
320-
321314
/**
322315
* Moves the feedback session to the recycle bin.
323316
*/

src/web/app/pages-instructor/instructor-session-base-page.component.ts

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -172,16 +172,6 @@ export abstract class InstructorSessionBasePageComponent {
172172
}, (resp: ErrorMessageOutput) => { this.statusMessageService.showErrorToast(resp.error.message); });
173173
}
174174

175-
/**
176-
* Edits the feedback session.
177-
*/
178-
editSession(model: SessionsTableRowModel): void {
179-
this.navigationService.navigateByURLWithParamEncoding(
180-
this.router,
181-
'/web/instructor/sessions/edit',
182-
{ courseid: model.feedbackSession.courseId, fsname: model.feedbackSession.feedbackSessionName });
183-
}
184-
185175
/**
186176
* Creates list of copy session requests from params
187177
* @param model the source session model

src/web/app/pages-instructor/instructor-sessions-page/instructor-sessions-page.component.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030
[courseCandidates]="courseCandidates" [columnsToShow]="[ SessionsTableColumn.COURSE_ID ]"
3131
[headerColorScheme]="SessionsTableHeaderColorScheme.BLUE"
3232
(sortSessionsTableRowModelsEvent)="sortSessionsTableRowModelsEvent($event)"
33-
(loadResponseRateEvent)="loadResponseRateEventHandler($event)" (editSessionEvent)="editSessionEventHandler($event)"
33+
(loadResponseRateEvent)="loadResponseRateEventHandler($event)"
3434
(moveSessionToRecycleBinEvent)="moveSessionToRecycleBinEventHandler($event)" (copySessionEvent)="copySessionEventHandler($event)"
3535
(submitSessionAsInstructorEvent)="submitSessionAsInstructorEventHandler($event)" (viewSessionResultEvent)="viewSessionResultEventHandler($event)"
3636
(publishSessionEvent)="publishSessionEventHandler($event)" (unpublishSessionEvent)="unpublishSessionEventHandler($event)"

src/web/app/pages-instructor/instructor-sessions-page/instructor-sessions-page.component.ts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -436,13 +436,6 @@ export class InstructorSessionsPageComponent extends InstructorSessionModalPageC
436436
this.loadResponseRate(this.sessionsTableRowModels[rowIndex]);
437437
}
438438

439-
/**
440-
* Edits the feedback session.
441-
*/
442-
editSessionEventHandler(rowIndex: number): void {
443-
this.editSession(this.sessionsTableRowModels[rowIndex]);
444-
}
445-
446439
/**
447440
* Restores a recycle bin feedback session.
448441
*/

0 commit comments

Comments
 (0)