Skip to content

e2e test: use encryption tab in enableKeyBackup instead of security & settings tab#29234

Merged
florianduros merged 3 commits into
developfrom
florianduros/enable-key-backup
Mar 27, 2025
Merged

e2e test: use encryption tab in enableKeyBackup instead of security & settings tab#29234
florianduros merged 3 commits into
developfrom
florianduros/enable-key-backup

Conversation

@florianduros
Copy link
Copy Markdown
Member

@florianduros florianduros commented Feb 10, 2025

Task #26468
Key backup section will be removed in #29088. Use the encryption tab instead of the security & privacy tab

@florianduros florianduros added the T-Task Tasks for the team like planning label Feb 10, 2025
@florianduros florianduros force-pushed the florianduros/enable-key-backup branch from 9a033cd to e1a7ade Compare March 26, 2025 13:19
@florianduros florianduros force-pushed the florianduros/enable-key-backup branch from e1a7ade to 5a0bc2d Compare March 26, 2025 13:58
@florianduros florianduros changed the title test(e2e crypto): use encryption tab in enableKeyBackup instead of security & settings tab e2e test: use encryption tab in enableKeyBackup instead of security & settings tab Mar 26, 2025
@florianduros florianduros marked this pull request as ready for review March 26, 2025 14:17
@florianduros florianduros requested a review from a team as a code owner March 26, 2025 14:17
@florianduros florianduros requested a review from richvdh March 26, 2025 14:17
Copy link
Copy Markdown
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread playwright/e2e/crypto/utils.ts Outdated
*
* Returns the recovery key
*/
export async function enableKeyBackup(app: ElementAppPage): Promise<string> {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should probably be renamed enableRecovery, but that doesn't have to happen right now

@florianduros florianduros enabled auto-merge March 27, 2025 11:23
@florianduros florianduros added this pull request to the merge queue Mar 27, 2025
Merged via the queue into develop with commit a5673f6 Mar 27, 2025
@florianduros florianduros deleted the florianduros/enable-key-backup branch March 27, 2025 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-Task Tasks for the team like planning

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants