185: second and third modals when clicking on save button

[aulamber]

Checklist

• Tests written for new code (and old code if feasible)
• Linter and other CI checks pass
• Sign-off given on the changes (see CONTRIBUTING.md)

[estellecomment]

Testé sur la review app :

• traduction manquante :

• le lien "en savoir plus" pointe vers la page de register. C'est volontaire ?

• sur le modal suivant, "Clés Tchap sauvegardées", le lien "en savoir plus" pointe aussi vers la page de register.

[estellecomment]

Il y a un patch rename-exported-keys qui modifie le fichier ExportE2eKeysDialog.tsx (même probleme que la derniere fois)

Tu peux voir qu'il y a un patch si tu travailles avec un env avec yarn-linked-dependencies : une fois les patches appliqués (c'est fait avec yarn postinstall), tu peux voir que le fichier ExportE2eKeysDialog.tsx a des lignes modifiées. Donc c'est qu'un patch l'a modifié.

[odelcroi]

• decide weither it is a regression or not (fix it now or file an issue) :

I found an expected behaviour (maybe not so dramatic let's see together)

• In a logged session
• Click on settings>privacy>export keys
• Go on with the procedure
• At the end there is a logout button whereas my intention was not to logout, only to export my keys

In the actual version, when you export your keys from this panel, you are not invited to logout, the panel close and you stay in the logged session.

[odelcroi]

• UX enhancement : devrait-on rappeler le nom du fichier dans cet écran ?

Bonjour,
Suite à votre message ci dessous, j'ai commencé à sauvegarder mes clés tchap, mais je ne les retrouve plus. Désolé, comment pouvez vous m'aider ?
Merci

[estellecomment]

Review app is not there, probably because there was inactivity for too long so I got deleted.
Closing and reopening the PR to trigger the re-creation of the review app, so that I can review.

[estellecomment]

• As we talked about together : one of the modals stays up after logout. It looks to me that it doesn't block the feature or create major misunderstanding or introduce a security problem, so I suggest filing an issue separately for this, to avoid delaying this PR.

[estellecomment]

• UX enhancement : devrait-on rappeler le nom du fichier dans cet écran ?

It's called "tchap-keys.txt" in general, but when we save the file multiple times, it's called "tchap-keys(2).txt" (on macos for example). So we don't know the exact name of the file, the browser did the renaming.

[odelcroi]

It's called "tchap-keys.txt" in general, but when we save the file multiple times, it's called "tchap-keys(2).txt" (on macos for example). So we don't know the exact name of the file, the browser did the renaming.

For better understanding, the "name in general" can be added and highlighted in the dialog text.

for instance : "... sauvegardées avec succès dans le fichier tchap-keys.txt..."