Merge pull request #2032 from bcgov/feature/ALCS-2431-remove-message-…

…flash-QA ALCS-2431 Prevents validation flash
bcgov · Dec 18, 2024 · ae6842a · ae6842a
2 parents d87af10 + 514953f
commit ae6842a
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 5 deletions.
diff --git a/...tion-types/decision-condition-types-dialog/decision-condition-types-dialog.component.html b/...tion-types/decision-condition-types-dialog/decision-condition-types-dialog.component.html
@@ -47,7 +47,7 @@ <h4>{{ isEdit ? 'Edit' : 'Create' }} Decision Condition Type</h4>
           <div>
             <mat-checkbox formControlName="isAdministrativeFeeAmountChecked">Administrative Fee Amount</mat-checkbox>
             <app-error-message
-              *ngIf="!conditionTypeForm.controls['isAdministrativeFeeAmountChecked'].valid"
+              *ngIf="!conditionTypeForm.controls['isAdministrativeFeeAmountChecked'].valid && !isLoading"
               [message]="'Field contains data, cannot be removed'"
             ></app-error-message>
           </div>
@@ -82,7 +82,7 @@ <h4>{{ isEdit ? 'Edit' : 'Create' }} Decision Condition Type</h4>
           <div>
             <mat-checkbox formControlName="isDateChecked"> Date </mat-checkbox>
             <app-error-message
-              *ngIf="!conditionTypeForm.controls['isDateChecked'].valid"
+              *ngIf="!conditionTypeForm.controls['isDateChecked'].valid && !isLoading"
               [message]="'Field contains data, cannot be removed'"
             ></app-error-message>
           </div>
@@ -122,7 +122,7 @@ <h4>{{ isEdit ? 'Edit' : 'Create' }} Decision Condition Type</h4>
           <div>
             <mat-checkbox formControlName="isSecurityAmountChecked">Security Amount</mat-checkbox>
             <app-error-message
-              *ngIf="!conditionTypeForm.controls['isSecurityAmountChecked'].valid"
+              *ngIf="!conditionTypeForm.controls['isSecurityAmountChecked'].valid && !isLoading"
               [message]="'Field contains data, cannot be removed'"
             ></app-error-message>
           </div>

diff --git a/...dition-types/decision-condition-types-dialog/decision-condition-types-dialog.component.ts b/...dition-types/decision-condition-types-dialog/decision-condition-types-dialog.component.ts
@@ -47,6 +47,7 @@ export class DecisionConditionTypesDialogComponent {
     @Inject(MAT_DIALOG_DATA) public data: DecisionDialogDataInterface | undefined,
     private dialogRef: MatDialogRef<DecisionConditionTypesDialogComponent>,
   ) {
+    this.isLoading = true;
     this.service = data?.service;
     this.conditionService = data?.conditionService;
     this.isEdit = !!data?.content;
@@ -111,6 +112,7 @@ export class DecisionConditionTypesDialogComponent {
 
     this.conditionTypeForm.get('isComponentToConditionChecked')?.disable();
     this.conditionTypeForm.get('isDescriptionChecked')?.disable();
+    this.isLoading = false;
   }
 
   ngOnInit(): void {
@@ -144,6 +146,9 @@ export class DecisionConditionTypesDialogComponent {
     if (this.conditionTypeForm.get('administrativeFeeAmount')?.value !== '') {
       dto.administrativeFeeAmount = this.conditionTypeForm.get('administrativeFeeAmount')?.value;
     }
+    if (!dto.isAdministrativeFeeAmountChecked) {
+      dto.administrativeFeeAmount = null;
+    }
     if (!this.service) return;
     if (this.isEdit) {
       await this.service.update(dto.code, dto);
@@ -158,11 +163,15 @@ export class DecisionConditionTypesDialogComponent {
     return (control: AbstractControl): Observable<{ [key: string]: any } | null> => {
       return of(control.value).pipe(
         debounceTime(300),
-        switchMap(() => {
+        switchMap(async () => {
           if (!this.conditionService) {
             throw Error('Condition service not found');
           }
-          return this.conditionService.fetchByTypeCode(this.conditionTypeForm.controls['code'].value);
+          // prevents the message flash by setting isLoading while fetching codes.
+          this.isLoading = true;
+          const result = await this.conditionService.fetchByTypeCode(this.conditionTypeForm.controls['code'].value);
+          this.isLoading = false;
+          return result;
         }),
         map((conditions) =>
           {