-
-
Notifications
You must be signed in to change notification settings - Fork 2.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
Add Recovery
section in the new user settings Encryption
tab
#28673
Merged
Merged
Changes from 1 commit
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
3e77b3d
Refine `SettingsSection` & `SettingsTab`
florianduros ae623f8
Add encryption tab
florianduros f9e48b4
Add recovery section
florianduros 0057f57
Add device verification
florianduros bb507b0
Rename `Panel` into `State`
florianduros 1aace3f
Update & add tests to user settings common
florianduros 70c084e
Add tests to `RecoveryPanel`
florianduros 7193998
Add tests to `ChangeRecoveryKey`
florianduros fec324e
Update CreateSecretStorageDialog-test snapshot
florianduros 44c6bce
Add tests to `EncryptionUserSettingsTab`
florianduros 075f6dc
Update existing screenshots of e2e tests
florianduros 895ad88
Add new encryption tab ownership to `@element-hq/element-crypto-web-r…
florianduros ba032a7
Add e2e tests
florianduros 7909ac9
Fix monospace font and add figma link to hardcoded value
florianduros 618557c
Add unit to Icon
florianduros c805cd8
Merge branch 'develop' into florianduros/encryption-tab
florianduros 7a372f7
Merge branch 'develop' into florianduros/encryption-tab
florianduros b20579d
Improve e2e doc
florianduros 24c537c
Assert that the crypto module is defined
florianduros 72adfa5
Add classname doc
florianduros 36c7e0e
Fix typo
florianduros 52076f1
Use `good` state instead of default
florianduros a0d904e
Rename `ChangeRecoveryKey.isSetupFlow` into `ChangeRecoveryKey.userHa…
florianduros 1a0e6dc
Move `deleteCachedSecrets` fixture in `recovery.spec.ts`
florianduros 0b254e5
Use one callback instead of two in `RecoveryPanel`
florianduros 6f236bd
Fix docs and naming of `utils.createBot`
florianduros 82bf2cc
Fix typo in `RecoveryPanel`
florianduros 84d11f8
Add more doc to the state of the `EncryptionUserSettingsTab`
florianduros 2fe5555
Rename `verification_required` into `set_up_encryption`
florianduros 4b365ba
Merge branch 'develop' into florianduros/encryption-tab
florianduros 8a9291a
Update test
florianduros 1c00502
ADd new license
florianduros dc940f5
Update comments and doc
florianduros 521cebf
Assert that `recoveryKey.encodedPrivateKey` is always defined
florianduros 7af44cc
Add comments to explain how the secrets could be uncached
florianduros 8bd5d6a
Use `matrixClient.secretStorage.getDefaultKeyId` instead of `matrixCl…
florianduros 086f28e
Merge branch 'develop' into florianduros/encryption-tab
florianduros 0c18708
Update existing screenshot to add encryption tab.
florianduros 0a52b7c
Update tests
florianduros e5dea48
Use new labels when changing the recovery key
florianduros f78c27a
Fix docs
florianduros bef3165
Don't reset key backup when creating a recovery key
florianduros ae163d1
Fix doc
florianduros ee4f3a8
Merge branch 'develop' into florianduros/encryption-tab
florianduros File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Binary file modified
BIN
-2.19 KB
(93%)
...ings/encryption-user-tab/recovery.spec.ts/change-key-1-encryption-tab-linux.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file modified
BIN
-227 Bytes
(99%)
...ings/encryption-user-tab/recovery.spec.ts/change-key-2-encryption-tab-linux.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I have to say, I wonder if it wouldn't be simpler to just have three states (
inform_user
,save_key
,confirm_key
) and distinguish between the two flows by looking at theuserHasRecoveryKey
prop.On the other hand, we should probably just get on and land this rather than messing with it forever. A thought for the future though.