Skip to content

UX/UI: Amélioration du message affiché avant de retirer un collaborateur [GEN-1823]#5386

Merged
tonial merged 1 commit into
masterfrom
alaurent/remove_member_modal
Jan 16, 2025
Merged

UX/UI: Amélioration du message affiché avant de retirer un collaborateur [GEN-1823]#5386
tonial merged 1 commit into
masterfrom
alaurent/remove_member_modal

Conversation

@tonial
Copy link
Copy Markdown
Contributor

@tonial tonial commented Jan 14, 2025

🤔 Pourquoi ?

Je n'ai pas ajouté de test sur le wording pour le moment, je ne sais pas si c'est très utile, à part pour s'assurer que le bon message s'affiche pour les prescripteurs.
Un avis ?

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ 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

@tonial tonial added the modifié Modifié dans le changelog. label Jan 14, 2025
@tonial tonial self-assigned this Jan 14, 2025
Copy link
Copy Markdown
Contributor

@EwenKorr EwenKorr left a comment

Choose a reason for hiding this comment

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

Il y a un beau mélange de "utilisateur" et "collaborateur", il y a peut-être moyen d'harmoniser ?

Et sinon, je n'ai pas d'avis pour les tests de formulation/choix de mots/wording 🤐

Comment thread itou/templates/includes/deactivate_member.html Outdated
Comment thread itou/templates/includes/deactivate_member.html Outdated
Comment thread itou/templates/includes/deactivate_member.html Outdated
@tonial tonial force-pushed the alaurent/remove_member_modal branch from 6e7af0e to 53dfbd2 Compare January 14, 2025 15:14
@tonial tonial added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Jan 14, 2025
@tonial tonial force-pushed the alaurent/remove_member_modal branch 2 times, most recently from fac8caa to ea2265d Compare January 14, 2025 15:21
@tonial
Copy link
Copy Markdown
Contributor Author

tonial commented Jan 14, 2025

J'ai ajouté une variable org_display pour savoir si il faut dire organisation ou structure en fonction du type d'utilisateur.

@tonial tonial added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Jan 14, 2025
@github-actions
Copy link
Copy Markdown

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

Comment thread itou/templates/includes/deactivate_member.html Outdated
Copy link
Copy Markdown
Member

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

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

Est-ce que ça ne vaudrait pas aussi le coup de faire un tour sur admin_role (retirer les droits administrateur) pour mettre en cohérence ?

@tonial tonial force-pushed the alaurent/remove_member_modal branch 2 times, most recently from 2b3af9a to 791080a Compare January 14, 2025 21:13
@tonial tonial added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Jan 14, 2025
@github-actions
Copy link
Copy Markdown

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

@tonial tonial force-pushed the alaurent/remove_member_modal branch from 791080a to 8ca8899 Compare January 16, 2025 09:03
@tonial tonial force-pushed the alaurent/remove_member_modal branch from 8ca8899 to 5c4cdaa Compare January 16, 2025 09:56
@tonial tonial added this pull request to the merge queue Jan 16, 2025
Merged via the queue into master with commit a90d094 Jan 16, 2025
@tonial tonial deleted the alaurent/remove_member_modal branch January 16, 2025 11:14
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