-
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-8252] Use new user-verification for exports on all clients #9244
[PM-8252] Use new user-verification for exports on all clients #9244
Conversation
…ExportComponent Add "@bitwarden/auth" as dependency to the vault-export-ui package New submit logic also checks for password-encrypted exports which will be need for future UI updates on browser and desktop
…he new UI is opened as a self-contained dialog
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9244 +/- ##
==========================================
+ Coverage 28.07% 28.09% +0.01%
==========================================
Files 2435 2435
Lines 70528 70491 -37
Branches 13143 13130 -13
==========================================
Hits 19804 19804
+ Misses 49185 49148 -37
Partials 1539 1539 ☔ View full report in Codecov by Sentry. |
No New Or Fixed Issues Found |
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.
LGTM
* Move/replace submit and userVerification logic from web into the BaseExportComponent Add "@bitwarden/auth" as dependency to the vault-export-ui package New submit logic also checks for password-encrypted exports which will be need for future UI updates on browser and desktop * Remove import/passing of the unneeded UserVerificationService * Remove app-user-verification from browser and desktop components as the new UI is opened as a self-contained dialog --------- Co-authored-by: Daniel James Smith <djsmith85@users.noreply.github.com>
Type of change
Objective
On the web we use the new
UserVerificationDialogComponent
which supports additional verification methods besides a user’s master password.This possibility is currently missing on browser and desktop, which fails for user’s that login with for example a
passkey
orLogin with Device
This also pays forward on the initiative to create a shareable export component which can be used across clients which would also enable password-encrypted exports to browser and desktop.
Code changes
Screenshots
Before
After
Before you submit