bitwarden/browser
1
0
Fork 0
mirror of https://github.com/bitwarden/browser synced 2025-12-16 16:23:44 +00:00
Code Issues Releases Wiki Activity

refactor submit logic

Browse Source
This commit is contained in:
rr-bw
2024-09-10 13:27:40 -07:00
parent 7d8886e4dc
commit feb16c3fd2
1 changed files with 11 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
libs/auth/src/angular/login/login.component.ts
View File

@@ -190,25 +190,19 @@ export class LoginComponentV2 implements OnInit, OnDestroy {
this.loginEmailService.clearValues(); this.loginEmailService.clearValues();
await this.router.navigate(["update-temp-password"]); await this.router.navigate(["update-temp-password"]);
} else { } else {
// Browser/Desktop specific (start) if (this.clientType === ClientType.Web) {
if (this.clientType === ClientType.Browser || this.clientType === ClientType.Desktop) { await this.goAfterLogIn(authResult.userId);
await this.syncService.fullSync(true); // TODO-rr-bw: check -> browser uses `await` and desktop uses `return`. Why?
}
// Browser/Desktop specific (end)
// Web specific (start)
await this.goAfterLogIn(authResult.userId);
// Web specific (end)
// Browser/Desktop (start)
this.loginEmailService.clearValues();
if (this.clientType === ClientType.Browser) {
await this.router.navigate(["/tabs/vault"]);
} else { } else {
await this.router.navigate(["vault"]); await this.syncService.fullSync(true); // TODO-rr-bw: browser used `await`, desktop used `return`. Why?
this.loginEmailService.clearValues();
if (this.clientType === ClientType.Browser) {
await this.router.navigate(["/tabs/vault"]);
} else {
await this.router.navigate(["vault"]); // Desktop
}
} }
// Browser/Desktop (end)
} }
}; };