In invite dialog add text indicating that we can invite multiple people #447

aulamber · 2023-03-02T10:47:03Z

Cf. issue #342

odelcroi · 2023-03-02T14:47:41Z

src/i18n/strings/tchap_translations.json

+    "en": "Invite someone using their name, username (like <userId/>) or <a>share this room</a>. It is possible to invite multiple people by copy-pasting an email list separated by a comma: prenom1.nom1@beta.gouv.fr,prenom2.nom2@beta.gouv.fr,prenom3.nom3@beta.gouv.fr."
+  },
+  "Invite someone using their name, username (like <userId/>) or <a>share this space</a>.": {
+    "fr": "Invitez quelqu’un grâce à son nom, nom d’utilisateur (tel que <userId/>) ou <a>partagez cet espace</a>. Il est possible d'inviter en masse en copiant et collant une liste d'emails séparé par une virgule : prenom1.nom1@beta.gouv.fr,prenom2.nom2@beta.gouv.fr,prenom3.nom3@beta.gouv.fr.",


odelcroi · 2023-03-02T14:48:57Z

src/i18n/strings/tchap_translations.json

@@ -722,5 +722,13 @@
  "We'll generate a Security Key for you to store somewhere safe, like a password manager or a safe.": {
    "fr": "Nous génèrerons une clé de récupération que vous devrez stocker dans un endroit sûr, comme un gestionnaire de mots de passe ou un coffre.",
    "en": "We'll generate a Recovery Key for you to store somewhere safe, like a password manager or a safe."
+  },
+  "Invite someone using their name, username (like <userId/>) or <a>share this room</a>.": {
+    "fr": "Invitez quelqu’un à partir de son nom, pseudo (comme <userId/>) ou <a>partagez ce salon</a>. Il est possible d'inviter en masse en copiant et collant une liste d'emails séparé par une virgule : prenom1.nom1@beta.gouv.fr,prenom2.nom2@beta.gouv.fr,prenom3.nom3@beta.gouv.fr.",


d'emails séparés

odelcroi · 2023-03-02T14:49:44Z

Can you provide a screenshot of the updated screen?

odelcroi · 2023-03-03T14:46:16Z

It also work if you copy paste the content of a csv file

jdauphant · 2023-03-07T09:27:11Z

...-dialog-add-text-indicating-that-we-can-invite-multiple-people/matrix-react-sdk+3.63.0.patch

+-        overflow-y: auto;
+        overflow: auto;
+         display: flex;
+         flex-wrap: wrap;
+
+@@ -194,6 +194,7 @@ limitations under the License.
+
+     .mx_InviteDialog_addressBar {
+         margin-inline-end: 0;
+        overflow: auto;


C'est une modification qui peut profiter à Element ? où c'est juste pour nous ?

Ça leur serait utile à eux aussi, leur version est également buggée. Cf:

Du coup, je peux quand-même merger mon bugfix chez nous ou pas ?

jdauphant · 2023-03-07T09:43:16Z

src/i18n/strings/tchap_translations.json

@@ -722,5 +722,13 @@
  "We'll generate a Security Key for you to store somewhere safe, like a password manager or a safe.": {
    "fr": "Nous génèrerons une clé de récupération que vous devrez stocker dans un endroit sûr, comme un gestionnaire de mots de passe ou un coffre.",
    "en": "We'll generate a Recovery Key for you to store somewhere safe, like a password manager or a safe."
+  },
+  "Invite someone using their name, email address, username (like <userId/>) or <a>share this room</a>.": {
+    "fr": "Invitez quelqu’un via son nom, e-mail ou pseudo (p. ex. <userId/>) ou <a>partagez ce salon</a>. Il est possible d'inviter en masse en copiant et collant une liste d'emails séparés par une virgule (prenom1.nom1@beta.gouv.fr, prenom2.nom2@beta.gouv.fr, prenom3.nom3@beta.gouv.fr) ou séparés par un saut à la ligne.",


@aulamber on n'a moyen de mettre des sauts de ligne ?

Je n'y arrive pas en modifiant la string de traduction seulement. Par contre je peux faire un patch dans le code d'Element pour rajouter une div... mais je trouve que c'est lourd juste pour un saut à la ligne, qu'en penses-tu ?

jdauphant

C'est bon pour la partie tech.
2 points :

Si le overflow est utile a Element, on peut faire la PR en // (comme ça on supprime le patch dans nos prochaines version)
Un relecture par @Caroline-lawson ou @areox-net pour la mise en forme et le texte

jdauphant · 2023-03-07T09:46:52Z

...-dialog-add-text-indicating-that-we-can-invite-multiple-people/matrix-react-sdk+3.63.0.patch

+@@ -38,7 +38,7 @@ limitations under the License.
+         min-height: 25px;
+         padding-inline-start: $spacing-8;
+         overflow-x: hidden;


du coup le "overflow-x: hidden;" devient inutile ?

Je l'ai enlevé, tu as raison

…can-invite-multiple-people

Caroline-lawson · 2023-03-09T09:04:48Z

Dans Tchap on utilise "mail" plutôt que "e-mail"

aulamber · 2023-03-10T10:22:31Z

Dans Tchap on utilise "mail" plutôt que "e-mail"

Modifié

In invite dialog add text indicating that we can invite multiple people

9ae3054

odelcroi reviewed Mar 2, 2023

View reviewed changes

aulamber added 2 commits March 3, 2023 11:32

Fix typo "separe"

24c2667

Add spaces between commas in label

5ec0eef

aulamber added 2 commits March 7, 2023 10:12

Add scrolling when copy-pasting a list and adequate texte with it

e87fe62

Add horizontal scroll for multiple emails

48063fa

aulamber requested a review from jdauphant March 7, 2023 09:22

jdauphant reviewed Mar 7, 2023

View reviewed changes

jdauphant requested review from areox-net and Caroline-lawson March 7, 2023 09:42

jdauphant reviewed Mar 7, 2023

View reviewed changes

jdauphant approved these changes Mar 7, 2023

View reviewed changes

jdauphant reviewed Mar 7, 2023

View reviewed changes

Merge branch 'develop_tchap' into 342-invite-dialog-indicate-that-we-…

99ec69f

…can-invite-multiple-people

aulamber closed this Mar 7, 2023

aulamber reopened this Mar 7, 2023

aulamber added 2 commits March 7, 2023 18:40

Fix breaking comma in patches from develop branch

4f6b89f

Remove an unecessary overflow property

94e97a4

areox-net approved these changes Mar 8, 2023

View reviewed changes

Remove e in email texts

0e230a6

aulamber merged commit b8a0630 into develop_tchap Mar 10, 2023

odelcroi mentioned this pull request Apr 4, 2023

Invite users to a room in Bulk via a file #272

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

In invite dialog add text indicating that we can invite multiple people #447

In invite dialog add text indicating that we can invite multiple people #447

aulamber commented Mar 2, 2023 •

edited

Loading

odelcroi Mar 2, 2023

odelcroi Mar 2, 2023

odelcroi commented Mar 2, 2023

odelcroi commented Mar 3, 2023

jdauphant Mar 7, 2023

aulamber Mar 8, 2023

aulamber Mar 8, 2023 •

edited

Loading

jdauphant Mar 7, 2023

aulamber Mar 8, 2023 •

edited

Loading

jdauphant left a comment

jdauphant Mar 7, 2023

aulamber Mar 8, 2023

Caroline-lawson commented Mar 9, 2023

aulamber commented Mar 10, 2023

In invite dialog add text indicating that we can invite multiple people #447

In invite dialog add text indicating that we can invite multiple people #447

Conversation

aulamber commented Mar 2, 2023 • edited Loading

odelcroi Mar 2, 2023

Choose a reason for hiding this comment

odelcroi Mar 2, 2023

Choose a reason for hiding this comment

odelcroi commented Mar 2, 2023

odelcroi commented Mar 3, 2023

jdauphant Mar 7, 2023

Choose a reason for hiding this comment

aulamber Mar 8, 2023

Choose a reason for hiding this comment

aulamber Mar 8, 2023 • edited Loading

Choose a reason for hiding this comment

jdauphant Mar 7, 2023

Choose a reason for hiding this comment

aulamber Mar 8, 2023 • edited Loading

Choose a reason for hiding this comment

jdauphant left a comment

Choose a reason for hiding this comment

jdauphant Mar 7, 2023

Choose a reason for hiding this comment

aulamber Mar 8, 2023

Choose a reason for hiding this comment

Caroline-lawson commented Mar 9, 2023

aulamber commented Mar 10, 2023

aulamber commented Mar 2, 2023 •

edited

Loading

aulamber Mar 8, 2023 •

edited

Loading

aulamber Mar 8, 2023 •

edited

Loading