Skip to content

Invitations : « Inviter un collaborateur » devient « Ajouter un collaborateur »#5934

Merged
celine-m-s merged 3 commits intomasterfrom
celinems/add-collaborator
Apr 23, 2025
Merged

Invitations : « Inviter un collaborateur » devient « Ajouter un collaborateur »#5934
celine-m-s merged 3 commits intomasterfrom
celinems/add-collaborator

Conversation

@celine-m-s
Copy link
Copy Markdown
Contributor

@celine-m-s celine-m-s commented Apr 10, 2025

🤔 Pourquoi ?

On doit désormais faire référence à un ajout de collaborateur et non plus à une invitation.

🍰 Comment ?

Changement de formulations.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

image

image

@celine-m-s celine-m-s self-assigned this Apr 10, 2025
@celine-m-s celine-m-s added the modifié Modifié dans le changelog. label Apr 10, 2025
@celine-m-s celine-m-s force-pushed the celinems/add-collaborator branch 3 times, most recently from 383552c to 0ab3e20 Compare April 10, 2025 14:39
@celine-m-s celine-m-s requested review from leo-naeka and tonial April 10, 2025 14:40
@celine-m-s celine-m-s added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Apr 10, 2025
@notion-workspace
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@celine-m-s celine-m-s force-pushed the celinems/add-collaborator branch from 0ab3e20 to cf41f51 Compare April 10, 2025 14:57
Copy link
Copy Markdown
Contributor

@tonial tonial left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍



def _toast_invitation_sent(invitations):
s = pluralizefr(len(invitations))
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@celine-m-s celine-m-s removed the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Apr 12, 2025
@tonial tonial added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Apr 14, 2025
@github-actions
Copy link
Copy Markdown

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

Copy link
Copy Markdown
Contributor

@leo-naeka leo-naeka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'était tel quel dans la carte, mais sauf erreur on met systématiquement "de" devant un adjectif précédant un nom indéfini

Comment thread itou/templates/signup/company_select.html Outdated
Comment thread itou/templates/signup/prescriber_check_already_exists.html Outdated
Comment thread itou/templates/signup/prescriber_check_already_exists.html Outdated
Comment thread itou/templates/signup/prescriber_request_invitation.html
@leo-naeka
Copy link
Copy Markdown
Contributor

Capture d’écran 2025-04-22 à 16 28 14

Les labels de ces deux colonnes se référaient à "Invitation" et ne collent plus.

À voir pour confirmation avec le métier, mais :

  1. Pour la première on peut remplacer par "Ajouté le" (ou bien "Ajout le" si on veut du non genré, mais bon ça se réfère à "collaborateur" ici...).
  2. Pour la seconde, deux solutions :
    • La déplacer après "Lien d'activation" (mais pas convaincu d'avoir le bouton copier en plein milieu…) :
      Capture d’écran 2025-04-22 à 16 36 10

    • Changer le label, par exemple :
      Capture d’écran 2025-04-22 à 16 38 12

@celine-m-s
Copy link
Copy Markdown
Contributor Author

Merci @leo-naeka ! Je propose Date d'ajout et Expiration du lien. J'en parle avec le métier.

@celine-m-s celine-m-s force-pushed the celinems/add-collaborator branch 2 times, most recently from cf79d59 to d400876 Compare April 22, 2025 18:27
@celine-m-s celine-m-s force-pushed the celinems/add-collaborator branch from d400876 to fefa502 Compare April 23, 2025 09:47
@celine-m-s celine-m-s enabled auto-merge April 23, 2025 09:51
@celine-m-s celine-m-s added this pull request to the merge queue Apr 23, 2025
Merged via the queue into master with commit 5863a7c Apr 23, 2025
14 checks passed
@celine-m-s celine-m-s deleted the celinems/add-collaborator branch April 23, 2025 10:06
@francoisfreitag francoisfreitag changed the title [Invitations] « Inviter un collaborateur » devient « Ajouter un collaborateur » Invitations : « Inviter un collaborateur » devient « Ajouter un collaborateur » Apr 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants