fix(Manage Students): Subscribe to retrieve config after remove user

#1449
WISE-Community · Oct 8, 2023 · 961c653 · 961c653
1 parent d1aa863
commit 961c653
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/...nitor/classroomMonitorComponents/manageStudents/manage-user/manage-user.component.spec.ts b/...nitor/classroomMonitorComponents/manageStudents/manage-user/manage-user.component.spec.ts
@@ -6,14 +6,15 @@ import { BrowserAnimationsModule } from '@angular/platform-browser/animations';
 import { ConfigService } from '../../../../services/configService';
 import { ManageUserComponent } from './manage-user.component';
 import { NO_ERRORS_SCHEMA } from '@angular/core';
+import { of } from 'rxjs';
 
 class ConfigServiceStub {
   getPermissions() {}
   getRunId() {
     return 123;
   }
   retrieveConfig() {
-    return {};
+    return of({});
   }
 }
 

diff --git a/...oomMonitor/classroomMonitorComponents/manageStudents/manage-user/manage-user.component.ts b/...oomMonitor/classroomMonitorComponents/manageStudents/manage-user/manage-user.component.ts
@@ -53,8 +53,13 @@ export class ManageUserComponent {
     this.http.delete(`/api/teacher/run/${runId}/student/${studentId}/remove`).subscribe({
       next: () => {
         this.removeUserEvent.emit(this.user);
-        this.snackBar.open($localize`Removed ${this.user.name} (${this.user.username}) from unit.`);
-        this.configService.retrieveConfig(`/api/config/classroomMonitor/${runId}`);
+        this.configService.retrieveConfig(`/api/config/classroomMonitor/${runId}`).subscribe({
+          next: () => {
+            this.snackBar.open(
+              $localize`Removed ${this.user.name} (${this.user.username}) from unit.`
+            );
+          }
+        });
       },
       error: () => {
         this.snackBar.open(