-
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
Auth/PM-7324 - Registration with Email Verification - Registration Start Component Implementation #9573
Auth/PM-7324 - Registration with Email Verification - Registration Start Component Implementation #9573
Conversation
… to API and trigger email.
…rification-registration-start-implementation + oss routing module merge conflict fixes.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9573 +/- ##
==========================================
- Coverage 28.72% 28.69% -0.03%
==========================================
Files 2497 2499 +2
Lines 72656 72766 +110
Branches 13542 13565 +23
==========================================
+ Hits 20867 20877 +10
- Misses 50187 50286 +99
- Partials 1602 1603 +1 ☔ View full report in Codecov by Sentry. |
…o desktop and browser extension
… by the email verification feature flag.
…ected by feature flag
…s to the new signup page.
…d passing undefined values into I18nService.t method
…eparately and merged to main.
…rification-registration-start-implementation + oss-routing.module merge conflict fix.
…or the browser extension.
… warning around BrowserAnimationsModule
…y email has something to land on.
…rification-registration-start-implementation + merge conflict fixes.
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.
Great work! Had a few questions
apps/web/src/app/auth/organization-invite/accept-organization.component.ts
Show resolved
Hide resolved
libs/auth/src/angular/registration/registration-start/registration-start-secondary.component.ts
Outdated
Show resolved
Hide resolved
libs/auth/src/angular/registration/registration-start/registration-start.component.ts
Outdated
Show resolved
Hide resolved
libs/auth/src/angular/registration/registration-start/registration-start.component.ts
Outdated
Show resolved
Hide resolved
…-start-implementation
🎟️ Tracking
PM-7324
Associated server PR: bitwarden/server#4173
📔 Objective
To implement the business logic of the first component in the new registration with email verification feature. This also includes redesigning the registration start component to have implicit consent to our privacy policy and terms of service while providing a mechanism for opting in or out to receiving marketing emails.
Tools & Admin Console team changes
/signup
route or the existing/register
route based on the email verification feature flag.📸 Screenshots
Storybook
Web
Note: the local environment is a special, JSON configured additional env. As such, it is not treated as a self hosted env, and the registration start component shows the receive marketing emails checkbox as it is only hidden on self hosted. This is only a web specific configuration inconsistency, but we didn't want to add conditionals to check if the env was dev and coerce it to treat it as self hosted so that it would be easier to locally test the receive marketing emails checkbox + putting dev env checks into prod code should probably be avoided.
Email Verification Required - New User
Note: in this scenario, new and existing users get the exact same experience to avoid account enumeration.
PM-7324.-.Web.-.Email.Verification.-.Registration.Start.-.Email.Verification.required.mov
Email Verification Not Required - New User
PM-7324.-.Web.-.Email.Verification.-.Registration.Start.-.Email.Verification.not.required.mov
Email Verification Not Required - Existing user
PM-7324.-.Web.-.Email.Verification.-.Registration.Start.-.Email.Verification.not.required.-.existing.user.mov
Browser Extension
Email Verification Required - New User
PM-7324.-.Browser.Extension.-.Email.Verification.-.Registration.Start.-.Email.Verification.required.mov
Email Verification Not Required - New User
PM-7324.-.Browser.Extension.-.Email.Verification.-.Registration.Start.-.Email.Verification.not.required.mov
Desktop
Email Verification Required - New User
PM-7324.-.Desktop.-.Email.Verification.-.Registration.Start.-.Email.Verification.required.mov
Email Verification Not Required - New User
PM-7324.-.Desktop.-.Email.Verification.-.Registration.Start.-.Email.Verification.not.required.mov
⏰ Reminders before review
🦮 Reviewer guidelines
:+1:
) or similar for great changes:memo:
) or ℹ️ (:information_source:
) for notes or general info:question:
) for questions:thinking:
) or 💭 (:thought_balloon:
) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion:art:
) for suggestions / improvements:x:
) or:warning:
) for more significant problems or concerns needing attention:seedling:
) or ♻️ (:recycle:
) for future improvements or indications of technical debt:pick:
) for minor or nitpick changes