-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PM-4951 Migrate Recover Two Factor Component #9170
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9170 +/- ##
=======================================
Coverage 28.16% 28.17%
=======================================
Files 2432 2432
Lines 71554 71557 +3
Branches 13376 13375 -1
=======================================
+ Hits 20151 20158 +7
+ Misses 49813 49810 -3
+ Partials 1590 1589 -1 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Just a few minor requests.
…951-migrate-recover-two-factor-component
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, just a few more minor updates and this is good to go.
…951-migrate-recover-two-factor-component
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For this page:
- Grab latest from
main
- Update the import in
oss-routing.module.ts
to:
import { AnonLayoutWrapperComponent } from "@bitwarden/auth/angular";
- Update the route to:
{
path: "",
component: AnonLayoutWrapperComponent,
children: [
{
path: "recover-2fa",
canActivate: [unauthGuardFn()],
children: [
{
path: "",
component: RecoverTwoFactorComponent,
},
{
path: "",
component: EnvironmentSelectorComponent,
outlet: "environment-selector",
},
],
data: {
pageTitle: "recoverAccountTwoStep",
titleId: "recoverAccountTwoStep",
},
},
],
},
You will need to import the web Environment Selector:
import { EnvironmentSelectorComponent } from "./components/environment-selector/environment-selector.component";
Type of change
Objective
Migrate the Recover Two Factor Component to use the component library.
Code changes
Screenshots
Recover.Two.Factor.-.Google.Chrome.2024-05-14.15-29-48.mp4