185: Add Save your Tchap Keys before logging out #291
Conversation
|
Are we missing some french translations? |
|
Les traductions sont a ajotuer dans |
|
|
||
| render() { | ||
| if (this.state.shouldLoadBackupStatus) { | ||
| // const description = ( |
There was a problem hiding this comment.
Je pense qu'on peut supprimer le code d'element-web dont on se sert pas... Parce qu'on a le fichier LogoutDialog qui existe tjs donc on peut faire de comparaisons facilement.
| {}, | ||
| { | ||
| b: (sub) => <b>{ sub }</b>, | ||
| requestLink: (sub) => ( |
There was a problem hiding this comment.
pas besoin de requestLink ici, parce que tu n'as pas mentionné de lien qui s'appelle "requestlink" dans le translated string.
| { dialogContent } | ||
| </BaseDialog> | ||
| ); | ||
| } else { |
There was a problem hiding this comment.
dans notre cas, on veut afficher le dialog complet dans tous les cas. On a pas besoin de ce "are you sure you want to sign out". Donc on peut enlever le if (this.state.shouldLoadBackupStatus) ligne 144. (et du coup enlever les fonctions pas utilisees au besoin)
|
Aussi à faire dans cette PR : le patch Et du coup il faudra supprimer ce patch (https://github.com/tchapgouv/tchap-web-v4/blob/2f9d72eea49f0bd73eec56f556dd94979a2ad4f7/patches/hide-secure-storage-2/matrix-react-sdk+3.54.0.patch) et l'enlever de patches.json ( tchap-web-v4/patches/patches.json Line 86 in 2f9d72e |
There was a problem hiding this comment.
Nice job ! :)
We will need an end-to-end test for this... I opened a separate issue. #302
(we'll have a single test for the whole flow I think)
| this.props.onFinished(true); | ||
| }; | ||
|
|
||
| helpUrl = "https://tchap.beta.gouv.fr/faq#tcq05_001"; |
There was a problem hiding this comment.
not used anywhere. Did you mean to add a link to this help page ?
Checklist