Add tab to download csv file of the list of users in a room #613

aulamber · 2023-06-01T14:15:36Z

Cf Issue #593

Before:

Proposal:

With tooltip:

FAQ page created:
https://aide.tchap.beta.gouv.fr/fr/article/comment-recuperer-les-membres-dun-salon-pour-les-inviter-sur-un-autre-salon-web-1gwsc6h/?bust=1687427559412

aulamber · 2023-06-01T14:22:45Z

@odelcroi
I have 2 proposals:

either have a simple tab which on click directly downloads the csv file, like this:

or on click open a popup like this:

If we choose the second option, I think we would need to offer to download the file in more than one extension (for instance csv and json), otherwise there is no need for this popup... what do you think? I can put either code on this PR, I've done both, but I think that the first option is the simplest and the best.

...es/add-tab-to-download-csv-file-of-the-list-of-users-in-a-room/matrix-react-sdk+3.71.1.patch

odelcroi · 2023-06-01T15:08:13Z

@aulamber Je pense que l'option 1 est suffisante.
J'ai testé d'exporter les users, ouvrir le fichier, copier coller dans un nouveau salon pour inviter les users.

Problème, étant d'extension csv, lorsque j'ouvre le fichier, il lance excel ce qui rend difficile le copier coller.

proposition : avec un fichie d'extension txt, le copier coller serait plus simple

aulamber · 2023-06-01T15:12:07Z

@aulamber Je pense que l'option 1 est suffisante. J'ai testé d'exporter les users, ouvrir le fichier, copier coller dans un nouveau salon pour inviter les users.

Problème, étant d'extension csv, lorsque j'ouvre le fichier, il lance excel ce qui rend difficile le copier coller.

proposition : avec un fichie d'extension txt, le copier coller serait plus simple

Ok, dans ce cas je transforme le csv en fichier .txt

aulamber · 2023-06-01T15:12:59Z

@aulamber Je pense que l'option 1 est suffisante. J'ai testé d'exporter les users, ouvrir le fichier, copier coller dans un nouveau salon pour inviter les users.

Problème, étant d'extension csv, lorsque j'ouvre le fichier, il lance excel ce qui rend difficile le copier coller.

proposition : avec un fichie d'extension txt, le copier coller serait plus simple

Chez moi il lance VSC, c'est un peu mieux, mais pas top non plus.

areox-net · 2023-06-01T15:34:33Z

@aulamber Je pense que l'option 1 est suffisante. J'ai testé d'exporter les users, ouvrir le fichier, copier coller dans un nouveau salon pour inviter les users.
Problème, étant d'extension csv, lorsque j'ouvre le fichier, il lance excel ce qui rend difficile le copier coller.
proposition : avec un fichie d'extension txt, le copier coller serait plus simple

Ok, dans ce cas je transforme le csv en fichier .txt

Très bien. Une liste en txt avec un click c'est top. On garde l'option 2 au cas où on nous demande d'autres formats.

aulamber · 2023-06-01T16:15:12Z

@aulamber Je pense que l'option 1 est suffisante. J'ai testé d'exporter les users, ouvrir le fichier, copier coller dans un nouveau salon pour inviter les users.
Problème, étant d'extension csv, lorsque j'ouvre le fichier, il lance excel ce qui rend difficile le copier coller.
proposition : avec un fichie d'extension txt, le copier coller serait plus simple

Ok, dans ce cas je transforme le csv en fichier .txt

Très bien. Une liste en txt avec un click c'est top. On garde l'option 2 au cas où on nous demande d'autres formats.

Hum, mais du coup, on affiche l'option 1 non, vu qu'on n'a qu'un type de format qui nous intéresse ? Pas l'option 2 avec la popup ? Je vois pas trop l'interet d'afficher une popup avec un multiselect à une seule option ;) @areox-net @Caroline-lawson

areox-net · 2023-06-01T16:17:27Z

@aulamber Je pense que l'option 1 est suffisante. J'ai testé d'exporter les users, ouvrir le fichier, copier coller dans un nouveau salon pour inviter les users.
Problème, étant d'extension csv, lorsque j'ouvre le fichier, il lance excel ce qui rend difficile le copier coller.
proposition : avec un fichie d'extension txt, le copier coller serait plus simple

Ok, dans ce cas je transforme le csv en fichier .txt

Très bien. Une liste en txt avec un click c'est top. On garde l'option 2 au cas où on nous demande d'autres formats.

Hum, mais du coup, on affiche l'option 1 non, vu qu'on n'a qu'un type de format qui nous intéresse ? Pas l'option 2 avec la popup ? Je vois pas trop l'interet d'afficher une popup avec un multiselect à une seule option ;) @areox-net @Caroline-lawson

Oui, on part sur l'option 1.
Je disais que l'option 2 pourrait etre utile dans le futur si le gens nous demandent un choix multiple de formats.

odelcroi · 2023-06-05T07:41:45Z

Est ce qu'on limite cette fonction aux admin du salons?

Caroline-lawson · 2023-06-05T07:46:34Z

Pourquoi est-ce qu'on la limiterait aux admins ? Je ne pense pas à des cas d'usage allant dans un sens ou dans l'autre. Tous les membres d'un salon voient les autres membres, donc je ne vois pas pourquoi ils ne pourraient pas tous utiliser la fonctionnalité.

aulamber · 2023-06-05T09:11:56Z

Pourquoi est-ce qu'on la limiterait aux admins ? Je ne pense pas à des cas d'usage allant dans un sens ou dans l'autre. Tous les membres d'un salon voient les autres membres, donc je ne vois pas pourquoi ils ne pourraient pas tous utiliser la fonctionnalité.

@odelcroi @Caroline-lawson : Du coup, on limite aux admins ou pas ? Moi je serais du même avis que Caroline, en quoi est-ce gênant d'autoriser les non-admins ? @jdauphant une opinion ?

jdauphant · 2023-06-05T14:50:46Z

Les données d'emails sont visibles par tous le monde dans les faits, il faut voir ça comme une chaîne d'email, tous le monde voit les emails des autres.

odelcroi · 2023-06-06T08:13:39Z

Les données d'emails sont visibles par tous le monde dans les faits, il faut voir ça comme une chaîne d'email, tous le monde voit les emails des autres.

et dans un forum public comme tchap info?
Par ailleurs la feature exporte les matrixId et non les mails
@aulamber pourrais-tu ajouter un screenshot du resultat?

Caroline-lawson · 2023-06-06T09:07:20Z

Est-ce que les users peuvent utiliser les matrix ID pour inviter les membres à un salon ?

S'ils doivent modifier chaque ID pour la transformer en adresse mail, ça perd de son intérêt, non ? L'idée c'est de rapidement pouvoir recréer un salon à partir des membres d'un salon existant.

aulamber · 2023-06-07T09:41:35Z

Les données d'emails sont visibles par tous le monde dans les faits, il faut voir ça comme une chaîne d'email, tous le monde voit les emails des autres.

et dans un forum public comme tchap info? Par ailleurs la feature exporte les matrixId et non les mails @aulamber pourrais-tu ajouter un screenshot du resultat?

@odelcroi Tu as raison ce ne sont pas des emails, car je ne les ai pas trouvés dans les données utilisateurs, ce sont les matrixIDs.

aulamber · 2023-06-07T09:42:53Z

Est-ce que les users peuvent utiliser les matrix ID pour inviter les membres à un salon ?

S'ils doivent modifier chaque ID pour la transformer en adresse mail, ça perd de son intérêt, non ? L'idée c'est de rapidement pouvoir recréer un salon à partir des membres d'un salon existant.

Oui j'ai testé, en copiant-collant les matrix IDs pour ajouter des utilisateurs dans une room, cela fonctionne, pas besoin de transformer en format d'email.

odelcroi · 2023-06-07T12:14:21Z

Tester sur (indiquer les versions) :

Firefox testé sur v113.0.1
Edge testé sur v114.0.1823.51
Chrome testé sur v113.0.5672.126
Safari testé sur v16.1

Caroline-lawson · 2023-06-07T13:13:18Z

Est-ce que les users peuvent utiliser les matrix ID pour inviter les membres à un salon ?
S'ils doivent modifier chaque ID pour la transformer en adresse mail, ça perd de son intérêt, non ? L'idée c'est de rapidement pouvoir recréer un salon à partir des membres d'un salon existant.

Oui j'ai testé, en copiant-collant les matrix IDs pour ajouter des utilisateurs dans une room, cela fonctionne, pas besoin de transformer en format d'email.

Ok super - il faudra s'assurer qu'on le précise bien aux users

aulamber · 2023-06-20T09:43:43Z

Est-ce que les users peuvent utiliser les matrix ID pour inviter les membres à un salon ?
S'ils doivent modifier chaque ID pour la transformer en adresse mail, ça perd de son intérêt, non ? L'idée c'est de rapidement pouvoir recréer un salon à partir des membres d'un salon existant.

Oui j'ai testé, en copiant-collant les matrix IDs pour ajouter des utilisateurs dans une room, cela fonctionne, pas besoin de transformer en format d'email.

Ok super - il faudra s'assurer qu'on le précise bien aux users

Ajouté un tooltip pour l'explication aux utilisateurs :

Caroline-lawson · 2023-06-20T14:25:01Z

La tooltip apparaît / disparaît en roll over ? Ou on pourrait la fermer avec un X en haut à) droite ?

On pourrait réduire le texte ? Par exemple :

En exportant les membres du salon, vous obtenez les identifiants des participants que vous pourrez copier/coller pour les inviter dans un autre salon.

odelcroi · 2023-06-07T12:12:35Z

src/tchap/i18n/strings/tchap_translations.json

@@ -934,5 +934,9 @@
  "Please only proceed if you're sure you've lost all of your other devices and your Security Key.": {
    "fr": "Veuillez ne continuer que si vous êtes certain d’avoir perdu tous vos autres appareils et votre Code de Récupération.",
    "en": "Please only proceed if you're sure you've lost all of your other devices and your Recovery Code."
+  },
+  "Export room members (txt)": {
+    "fr": "Exporter les membres du salon (txt)",


remplacer (txt) par csv qui est le format des données alors que txt est le format du fichier

aulamber · 2023-06-21T09:38:22Z

En exportant les membres du salon, vous obtenez les identifiants des participants que vous pourrez copier/coller pour les inviter dans un autre salon.

C'est une tooltip donc il ne peut pas y avoir de croix pour fermer. C'est juste au hover. Du coup, je cherche une autre solution avec croix ou on reste là-dessus @Caroline-lawson ?

J'ai bien modifié le texte par ailleurs.

odelcroi · 2023-06-21T14:18:18Z

src/tchap/i18n/strings/tchap_translations.json

+    "en": "By exporting the members of the room, you obtain the identifiers of the participants that you can copy/paste to invite them into another room."
+  },
+
+  "Destroy cross-signing keys?": {


ces label font-ils partir de cette PR?

Oui ils font bien partie de cette PR.

Caroline-lawson · 2023-06-22T08:51:24Z

En exportant les membres du salon, vous obtenez les identifiants des participants que vous pourrez copier/coller pour les inviter dans un autre salon.

C'est une tooltip donc il ne peut pas y avoir de croix pour fermer. C'est juste au hover. Du coup, je cherche une autre solution avec croix ou on reste là-dessus @Caroline-lawson ?

J'ai bien modifié le texte par ailleurs.

On peut rester sur le hover dans un premier temps. On pourra voir plus tard s'il y a une solution plus accessible @areox-net tu aurais des idées ?

aulamber · 2023-06-22T10:14:13Z

I add an FAQ page explaining this feature:
https://aide.tchap.beta.gouv.fr/fr/article/comment-recuperer-les-membres-dun-salon-pour-les-inviter-sur-un-autre-salon-web-1gwsc6h/?bust=1687427559412

* Add tab to download csv file of the list of users in a room * Put download file in tchap utils, and change csv to txt donwload * Add tooltip * Add tab to download csv file of the list of users in a room (#613) * Add tab to download csv file of the list of users in a room * Put download file in tchap utils, and change csv to txt donwload * Add tooltip * Add tests for button download txt file * Undo cypress modifs * Fix eslint * Add cypress test * Fix eslint * Cleanup * Move button to Members menu * Add back tooltip, using AccessibleTooltipButton * Move button to the top, with other button * Use FileDownloader from react-sdk rather than implement our own * Remove previous implementation * add room name in exported filename * Rename patch, shorter name * Update translations * Add icon (better icon coming) * Button styling * Final icon * Better names for things * Separate ids with commas in text file * Cypress test WIP * Cypress : use normal room name * Cypress tooltip test * Cypress : leave room when done * Rename file * Fit linter * Make separate component for minimal footprint in patch --------- Co-authored-by: Estelle Comment <estelle.comment@gmail.com>

Add tab to download csv file of the list of users in a room

3032915

aulamber requested a review from odelcroi June 1, 2023 14:22

odelcroi reviewed Jun 1, 2023

View reviewed changes

...es/add-tab-to-download-csv-file-of-the-list-of-users-in-a-room/matrix-react-sdk+3.71.1.patch Show resolved Hide resolved

Put download file in tchap utils, and change csv to txt donwload

f3ff280

aulamber closed this Jun 19, 2023

aulamber reopened this Jun 19, 2023

Add tooltip

71b0cbf

odelcroi approved these changes Jun 20, 2023

View reviewed changes

odelcroi self-requested a review June 20, 2023 15:36

Fix conflict and change wording

d52fa19

odelcroi reviewed Jun 21, 2023

View reviewed changes

aulamber merged commit 1e29119 into develop_tchap Jun 22, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add tab to download csv file of the list of users in a room #613

Add tab to download csv file of the list of users in a room #613

aulamber commented Jun 1, 2023 •

edited

Loading

aulamber commented Jun 1, 2023

odelcroi commented Jun 1, 2023 •

edited

Loading

aulamber commented Jun 1, 2023

aulamber commented Jun 1, 2023

areox-net commented Jun 1, 2023

aulamber commented Jun 1, 2023

areox-net commented Jun 1, 2023

odelcroi commented Jun 5, 2023

Caroline-lawson commented Jun 5, 2023 •

edited

Loading

aulamber commented Jun 5, 2023 •

edited by Caroline-lawson

Loading

jdauphant commented Jun 5, 2023

odelcroi commented Jun 6, 2023 •

edited

Loading

Caroline-lawson commented Jun 6, 2023

aulamber commented Jun 7, 2023 •

edited

Loading

aulamber commented Jun 7, 2023 •

edited

Loading

odelcroi commented Jun 7, 2023 •

edited by aulamber

Loading

Caroline-lawson commented Jun 7, 2023

aulamber commented Jun 20, 2023

Caroline-lawson commented Jun 20, 2023 •

edited

Loading

odelcroi Jun 7, 2023

aulamber commented Jun 21, 2023

odelcroi Jun 21, 2023

aulamber Jun 22, 2023

Caroline-lawson commented Jun 22, 2023

aulamber commented Jun 22, 2023

Add tab to download csv file of the list of users in a room #613

Add tab to download csv file of the list of users in a room #613

Conversation

aulamber commented Jun 1, 2023 • edited Loading

aulamber commented Jun 1, 2023

odelcroi commented Jun 1, 2023 • edited Loading

aulamber commented Jun 1, 2023

aulamber commented Jun 1, 2023

areox-net commented Jun 1, 2023

aulamber commented Jun 1, 2023

areox-net commented Jun 1, 2023

odelcroi commented Jun 5, 2023

Caroline-lawson commented Jun 5, 2023 • edited Loading

aulamber commented Jun 5, 2023 • edited by Caroline-lawson Loading

jdauphant commented Jun 5, 2023

odelcroi commented Jun 6, 2023 • edited Loading

Caroline-lawson commented Jun 6, 2023

aulamber commented Jun 7, 2023 • edited Loading

aulamber commented Jun 7, 2023 • edited Loading

odelcroi commented Jun 7, 2023 • edited by aulamber Loading

Caroline-lawson commented Jun 7, 2023

aulamber commented Jun 20, 2023

Caroline-lawson commented Jun 20, 2023 • edited Loading

odelcroi Jun 7, 2023

Choose a reason for hiding this comment

aulamber commented Jun 21, 2023

odelcroi Jun 21, 2023

Choose a reason for hiding this comment

aulamber Jun 22, 2023

Choose a reason for hiding this comment

Caroline-lawson commented Jun 22, 2023

aulamber commented Jun 22, 2023

aulamber commented Jun 1, 2023 •

edited

Loading

odelcroi commented Jun 1, 2023 •

edited

Loading

Caroline-lawson commented Jun 5, 2023 •

edited

Loading

aulamber commented Jun 5, 2023 •

edited by Caroline-lawson

Loading

odelcroi commented Jun 6, 2023 •

edited

Loading

aulamber commented Jun 7, 2023 •

edited

Loading

aulamber commented Jun 7, 2023 •

edited

Loading

odelcroi commented Jun 7, 2023 •

edited by aulamber

Loading

Caroline-lawson commented Jun 20, 2023 •

edited

Loading