[PM-2057] update two factor email dialog (#8974)

* migrating two factor email component

* two factor email component migration

* two factor email component migration

* two factor email component migration

apps/web/src/app/auth/settings/two-factor-email.component.html

@@ -1,28 +1,10 @@
-<div class="modal fade" role="dialog" aria-modal="true" aria-labelledby="2faEmailTitle">
+<form [formGroup]="formGroup" [bitSubmit]="submit" *ngIf="authed">
-  <div class="modal-dialog" role="document">
+  <bit-dialog>
-    <div class="modal-content">
+    <span bitDialogTitle>
-      <div class="modal-header">
-        <h1 class="modal-title" id="2faEmailTitle">
       {{ "twoStepLogin" | i18n }}
-          <small>{{ "emailTitle" | i18n }}</small>
+      <span bitTypography="body1">{{ "emailTitle" | i18n }}</span>
-        </h1>
+    </span>
-        <button
+    <ng-container bitDialogContent>
-          type="button"
-          class="close"
-          data-dismiss="modal"
-          appA11yTitle="{{ 'close' | i18n }}"
-        >
-          <span aria-hidden="true">&times;</span>
-        </button>
-      </div>
-      <form
-        #form
-        (ngSubmit)="submit()"
-        [appApiAction]="formPromise"
-        ngNativeValidate
-        *ngIf="authed"
-      >
-        <div class="modal-body">
       <ng-container *ngIf="enabled">
         <app-callout type="success" title="{{ 'enabled' | i18n }}" icon="bwi bwi-check-circle">
           {{ "twoStepLoginProviderEnabled" | i18n }}
@@ -30,72 +12,42 @@
         <strong>{{ "email" | i18n }}:</strong> {{ email }}
       </ng-container>
       <ng-container *ngIf="!enabled">
-            <p class="d-flex">
+        <p class="tw-flex">
-              <span class="mr-3">{{ "twoFactorEmailDesc" | i18n }}</span>
+          <span class="tw-mr-3">{{ "twoFactorEmailDesc" | i18n }}</span>
-              <img class="float-right ml-auto mfaType1" alt="Email logo" />
+          <img class="tw-float-right tw-ml-auto mfaType1" alt="Email logo" />
         </p>
-            <div class="form-group">
+        <bit-form-field>
-              <label for="email">1. {{ "twoFactorEmailEnterEmail" | i18n }}</label>
+          <bit-label>1. {{ "twoFactorEmailEnterEmail" | i18n }}</bit-label>
           <input
-                id="email"
+            bitInput
             type="text"
-                name="Email"
+            formControlName="email"
-                class="form-control"
-                [(ngModel)]="email"
-                required
             inputmode="email"
             appInputVerbatim="false"
           />
-            </div>
+        </bit-form-field>
-            <div class="mb-3 d-flex">
+        <div class="tw-mb-3 tw-flex">
-              <button
+          <button bitButton type="button" buttonType="primary" [bitAction]="sendEmail">
-                #sendBtn
+            {{ "sendEmail" | i18n }}
-                type="button"
-                class="btn btn-outline-primary btn-sm btn-submit align-self-start"
-                (click)="sendEmail()"
-                [appApiAction]="emailPromise"
-                [disabled]="$any(sendBtn).loading"
-              >
-                <i
-                  class="bwi bwi-spinner bwi-spin"
-                  title="{{ 'loading' | i18n }}"
-                  aria-hidden="true"
-                ></i>
-                <span>{{ "sendEmail" | i18n }}</span>
           </button>
-              <span class="text-success ml-3" *ngIf="sentEmail">
+          <span class="tw-text-success tw-ml-3" *ngIf="sentEmail">
             {{ "verificationCodeEmailSent" | i18n: sentEmail }}
           </span>
         </div>
-            <div class="form-group">
+        <bit-form-field>
-              <label for="token">2. {{ "twoFactorEmailEnterCode" | i18n }}</label>
+          <bit-label>2. {{ "twoFactorEmailEnterCode" | i18n }}</bit-label>
-              <input
+          <input bitInput type="text" formControlName="token" appInputVerbatim />
-                id="token"
+        </bit-form-field>
-                type="text"
-                name="Token"
-                class="form-control"
-                [(ngModel)]="token"
-                required
-                appInputVerbatim
-              />
-            </div>
       </ng-container>
-        </div>
+    </ng-container>
-        <div class="modal-footer">
+    <ng-container bitDialogFooter>
-          <button type="submit" class="btn btn-primary btn-submit" [disabled]="form.loading">
+      <button bitButton bitFormButton type="submit" buttonType="primary">
-            <i
-              class="bwi bwi-spinner bwi-spin"
-              title="{{ 'loading' | i18n }}"
-              aria-hidden="true"
-            ></i>
         <span *ngIf="!enabled">{{ "enable" | i18n }}</span>
         <span *ngIf="enabled">{{ "disable" | i18n }}</span>
       </button>
-          <button type="button" class="btn btn-outline-secondary" data-dismiss="modal">
+      <button bitButton bitFormButton type="button" buttonType="secondary" bitDialogClose>
         {{ "close" | i18n }}
       </button>
-        </div>
+    </ng-container>
-      </form>
+  </bit-dialog>
-    </div>
+</form>
-  </div>
-</div>

apps/web/src/app/auth/settings/two-factor-email.component.ts

@@ -1,4 +1,6 @@
-import { Component } from "@angular/core";
+import { DIALOG_DATA, DialogConfig, DialogRef } from "@angular/cdk/dialog";
+import { Component, EventEmitter, Inject, Output } from "@angular/core";
+import { FormBuilder, Validators } from "@angular/forms";
 import { firstValueFrom, map } from "rxjs";
 
 import { ApiService } from "@bitwarden/common/abstractions/api.service";
@@ -19,18 +21,22 @@ import { TwoFactorBaseComponent } from "./two-factor-base.component";
 @Component({
   selector: "app-two-factor-email",
   templateUrl: "two-factor-email.component.html",
+  outputs: ["onUpdated"],
 })
 export class TwoFactorEmailComponent extends TwoFactorBaseComponent {
+  @Output() onChangeStatus: EventEmitter<boolean> = new EventEmitter();
   type = TwoFactorProviderType.Email;
-  email: string;
-  token: string;
   sentEmail: string;
   formPromise: Promise<TwoFactorEmailResponse>;
   emailPromise: Promise<unknown>;
-
   override componentName = "app-two-factor-email";
+  formGroup = this.formBuilder.group({
+    token: [null],
+    email: ["", [Validators.email, Validators.required]],
+  });
 
   constructor(
+    @Inject(DIALOG_DATA) protected data: AuthResponse<TwoFactorEmailResponse>,
     apiService: ApiService,
     i18nService: I18nService,
     platformUtilsService: PlatformUtilsService,
@@ -38,6 +44,8 @@ export class TwoFactorEmailComponent extends TwoFactorBaseComponent {
     userVerificationService: UserVerificationService,
     private accountService: AccountService,
     dialogService: DialogService,
+    private formBuilder: FormBuilder,
+    private dialogRef: DialogRef,
   ) {
     super(
       apiService,
@@ -48,31 +56,49 @@ export class TwoFactorEmailComponent extends TwoFactorBaseComponent {
       dialogService,
     );
   }
+  get token() {
+    return this.formGroup.get("token").value;
+  }
+  set token(value: string) {
+    this.formGroup.get("token").setValue(value);
+  }
+  get email() {
+    return this.formGroup.get("email").value;
+  }
+  set email(value: string) {
+    this.formGroup.get("email").setValue(value);
+  }
+
+  async ngOnInit() {
+    await this.auth(this.data);
+  }
 
   auth(authResponse: AuthResponse<TwoFactorEmailResponse>) {
     super.auth(authResponse);
     return this.processResponse(authResponse.response);
   }
 
-  submit() {
+  submit = async () => {
     if (this.enabled) {
-      return super.disable(this.formPromise);
+      await this.disableEmail();
+      this.onChangeStatus.emit(false);
     } else {
-      return this.enable();
+      await this.enable();
+      this.onChangeStatus.emit(true);
     }
+  };
+
+  private disableEmail() {
+    return super.disable(this.formPromise);
   }
 
-  async sendEmail() {
+  sendEmail = async () => {
-    try {
     const request = await this.buildRequestModel(TwoFactorEmailRequest);
     request.email = this.email;
     this.emailPromise = this.apiService.postTwoFactorEmailSetup(request);
     await this.emailPromise;
     this.sentEmail = this.email;
-    } catch (e) {
+  };
-      this.logService.error(e);
-    }
-  }
 
   protected async enable() {
     const request = await this.buildRequestModel(UpdateTwoFactorEmailRequest);
@@ -86,6 +112,10 @@ export class TwoFactorEmailComponent extends TwoFactorBaseComponent {
     });
   }
 
+  onClose = () => {
+    this.dialogRef.close(this.enabled);
+  };
+
   private async processResponse(response: TwoFactorEmailResponse) {
     this.token = null;
     this.email = response.email;
@@ -96,4 +126,15 @@ export class TwoFactorEmailComponent extends TwoFactorBaseComponent {
       );
     }
   }
+  /**
+   * Strongly typed helper to open a TwoFactorEmailComponentComponent
+   * @param dialogService Instance of the dialog service that will be used to open the dialog
+   * @param config Configuration for the dialog
+   */
+  static open(
+    dialogService: DialogService,
+    config: DialogConfig<AuthResponse<TwoFactorEmailResponse>>,
+  ) {
+    return dialogService.open<boolean>(TwoFactorEmailComponent, config);
+  }
 }

apps/web/src/app/auth/settings/two-factor-setup.component.ts

@@ -1,3 +1,4 @@
+import { DialogRef } from "@angular/cdk/dialog";
 import { Component, OnDestroy, OnInit, Type, ViewChild, ViewContainerRef } from "@angular/core";
 import { firstValueFrom, lastValueFrom, Observable, Subject, takeUntil } from "rxjs";
 
@@ -178,9 +179,12 @@ export class TwoFactorSetupComponent implements OnInit, OnDestroy {
         if (!result) {
           return;
         }
-        const emailComp = await this.openModal(this.emailModalRef, TwoFactorEmailComponent);
+        const authComp: DialogRef<boolean, any> = TwoFactorEmailComponent.open(this.dialogService, {
-        await emailComp.auth(result);
+          data: result,
-        emailComp.onUpdated.pipe(takeUntil(this.destroy$)).subscribe((enabled: boolean) => {
+        });
+        authComp.componentInstance.onChangeStatus
+          .pipe(takeUntil(this.destroy$))
+          .subscribe((enabled: boolean) => {
             this.updateStatus(enabled, TwoFactorProviderType.Email);
           });
         break;