Skip to content

Extract ResetIdentityBody into a separate object to allow re-using it#29700

Merged
andybalaam merged 1 commit into
developfrom
andybalaam/reset-encryption-redesign
Apr 14, 2025
Merged

Extract ResetIdentityBody into a separate object to allow re-using it#29700
andybalaam merged 1 commit into
developfrom
andybalaam/reset-encryption-redesign

Conversation

@andybalaam
Copy link
Copy Markdown
Member

@andybalaam andybalaam commented Apr 9, 2025

Part of #29227

This should not change app behaviour at all - pure refactoring, so the existing tests should cover it.

This is mostly work by @uhoreg , rebased and refactored a bit here.

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 src/components/views/settings/encryption/ResetIdentityPanel.tsx Outdated
Comment thread src/components/views/settings/encryption/ResetIdentityBody.tsx Outdated
@andybalaam andybalaam force-pushed the andybalaam/reset-encryption-redesign branch from bdb1012 to f6f58a2 Compare April 11, 2025 15:01
@andybalaam andybalaam enabled auto-merge April 11, 2025 15:01
@andybalaam andybalaam added this pull request to the merge queue Apr 11, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 11, 2025
@andybalaam andybalaam added this pull request to the merge queue Apr 14, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 14, 2025
@andybalaam andybalaam added this pull request to the merge queue Apr 14, 2025
Merged via the queue into develop with commit 6b052fd Apr 14, 2025
32 checks passed
@andybalaam andybalaam deleted the andybalaam/reset-encryption-redesign branch April 14, 2025 08:00
snowping pushed a commit to Novaloop-AG/element-web that referenced this pull request Jun 22, 2025
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