Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Clarify logout key backup warning dialog #5363

Closed
wants to merge 1 commit into from
Closed

Clarify logout key backup warning dialog #5363

wants to merge 1 commit into from

Conversation

notramo
Copy link
Contributor

@notramo notramo commented Oct 28, 2020

Added details to the logout key backup warning dialog, why access to encrypted messages will be lost.
Closes element-hq/element-web#15565

Signed-off by: notramo [email protected]


Here's what your changelog entry will look like:

✨ Features

@turt2live
Copy link
Member

@notramo this needs sign off and a yarn i18n

Added details to the logout key backup warning dialog, why access to encrypted messages will be lost.
Closes element-hq/element-web#15565
@jryans
Copy link
Collaborator

jryans commented Oct 29, 2020

@notramo Hmm, seems like strings still don't match, did you run yarn i18n?

@notramo
Copy link
Contributor Author

notramo commented Oct 29, 2020

I don't know React or anything JS. It seemed trivial to patch it, but it is more complicated than I think. Could someone take it over?

@MadLittleMods MadLittleMods added T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements Z-Community-PR Issue is solved by a community member's PR labels Jun 2, 2022
@MadLittleMods
Copy link
Contributor

I don't know React or anything JS. It seemed trivial to patch it, but it is more complicated than I think. Could someone take it over?

I've re-created the PR based on the latest code ⏩ #8741

@notramo notramo deleted the patch-1 branch June 17, 2022 17:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements Z-Community-PR Issue is solved by a community member's PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Logout key backup warning is not clear
4 participants