feat(change-password-component): Change Password Update [18720] - Small changes and added some clarification on where I'm blocked

libs/auth/src/angular/change-password/change-password.component.html

@@ -9,7 +9,7 @@
   <auth-input-password
     [flow]="inputPasswordFlow"
     [email]="email"
-    [userId]="activeUserId"
+    [userId]="userId"
     [loading]="submitting"
     [masterPasswordPolicyOptions]="masterPasswordPolicyOptions"
     [inlineButtons]="true"

libs/auth/src/angular/change-password/change-password.component.ts

@@ -45,7 +45,7 @@ export class ChangePasswordComponent implements OnInit {
 
   activeAccount: Account | null = null;
   email!: string;
-  activeUserId?: UserId;
+  userId?: UserId;
   masterPasswordPolicyOptions?: MasterPasswordPolicyOptions;
   initializing = true;
   submitting = false;
@@ -73,23 +73,29 @@ export class ChangePasswordComponent implements OnInit {
       throw new Error("No active active account found while trying to change passwords.");
     }
 
-    this.activeUserId = this.activeAccount.id;
+    this.userId = this.activeAccount.id;
     this.email = this.activeAccount.email;
 
-    if (!this.activeUserId) {
+    if (!this.userId) {
       throw new Error("activeUserId not found");
     }
 
     this.masterPasswordPolicyOptions = await firstValueFrom(
-      this.policyService.masterPasswordPolicyOptions$(this.activeUserId),
+      this.policyService.masterPasswordPolicyOptions$(this.userId),
     );
 
+    /**
+     * In the event of the org invitation flow, this will always be ForceSetPasswordReason.None
+     * because the `password-login.strategy` short circuits before setting the force set password
+     * reason. We used to have two separate components, update-temp-password and update-password
+     * which could show discrete messages based on the flow, but we cannot do that with one shared
+     * component. I cannot use the AcceptOrganizationInviteService to determine if we have an org
+     * invite so how can I determine that?
+     */
     this.forceSetPasswordReason = await firstValueFrom(
-      this.masterPasswordService.forceSetPasswordReason$(this.activeUserId),
+      this.masterPasswordService.forceSetPasswordReason$(this.userId),
     );
 
-    this.initializing = false;
-
     if (this.forceSetPasswordReason === ForceSetPasswordReason.AdminForcePasswordReset) {
       this.anonLayoutWrapperDataService.setAnonLayoutWrapperData({
         pageIcon: LockIcon,
@@ -104,6 +110,8 @@ export class ChangePasswordComponent implements OnInit {
         maxWidth: "lg",
       });
     }
+
+    this.initializing = false;
   }
 
   async logOut() {
@@ -141,17 +149,17 @@ export class ChangePasswordComponent implements OnInit {
           passwordInputResult.newPasswordHint,
         );
       } else {
-        if (!this.activeUserId) {
+        if (!this.userId) {
           throw new Error("userId not found");
         }
 
         if (this.forceSetPasswordReason === ForceSetPasswordReason.AdminForcePasswordReset) {
           await this.changePasswordService.changePasswordForAccountRecovery(
             passwordInputResult,
-            this.activeUserId,
+            this.userId,
           );
         } else {
-          await this.changePasswordService.changePassword(passwordInputResult, this.activeUserId);
+          await this.changePasswordService.changePassword(passwordInputResult, this.userId);
         }
 
         this.toastService.showToast({

libs/auth/src/common/utilities/index.ts

@@ -1,2 +1 @@
 export * from "./decode-jwt-token-to-json.utility";
-export * from "./route-list";

libs/auth/src/common/utilities/route-list.ts

@@ -1,6 +0,0 @@
-export const RouteList = {
-  // AdminRecoveryChangePasswordRoute: "recovery-change-password",
-  // NonCompliantChangePasswordRoute: "change-password",
-  // OLDUpdateTempPassword: "update-temp-password",
-  // OLDNonCompliantPasswordOnLogin: "update-password",
-};