fix: reload experiment after default template update

The reload of the experiment was done in the wrong place. Sometimes the experiment template id subject signaled an update before the templateId was updated.
UST-QuAntiL · Aug 2, 2023 · d917a68 · d917a68
1 parent 3445eab
commit d917a68
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/app/components/experiment/experiment.component.ts b/src/app/components/experiment/experiment.component.ts
@@ -183,12 +183,11 @@ export class ExperimentComponent implements OnInit, OnDestroy {
         });
     }
 
-    async updateExperimentDefaultTemplate(templateId: string | null) {
+    updateExperimentDefaultTemplate(templateId: string | null) {
         if (this.experimentId == null) {
             console.warn("Experiment ID is null!");
             return;
         }
-        await this.templates.setExperimentDefaultTemplate(this.experimentId, templateId);
-        this.experimentService.reloadExperiment();
+        this.templates.setExperimentDefaultTemplate(this.experimentId, templateId);
     }
 }
diff --git a/src/app/services/templates.service.ts b/src/app/services/templates.service.ts
@@ -234,10 +234,11 @@ export class TemplatesService {
         return templateResponse?.data?.groups ?? [];
     }
 
-    async setExperimentDefaultTemplate(experimentId: string, templateId: string | null) {
+    setExperimentDefaultTemplate(experimentId: string, templateId: string | null) {
         this.backend.updateExperimentDefaultTemplate(experimentId, templateId).pipe(take(1)).subscribe(
             response => {
                 this.experimentTemplateIdSubject.next(response?.templateId ?? null);
+                this.currentExperiment.reloadExperiment();
             }
         );
     }