zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 161 forks source link

Changer l'auteur du MP d'ajout d'auteur #5777

Open Karnaj opened 4 years ago

Karnaj commented 4 years ago

Lorsqu'un membre est ajouté en auteur en tutoriel, un message privé lui est envoyé par Clem. Ce serait mieux que l'auteur du message soit le membre qui l'a ajouté au tutoriel.


Ça se passe dans la vue d'ajout d'auteur.

Situphen commented 4 years ago

Je ne suis franchement pas sûr que ce soit souhaitable, l'idée du compte Clem est de bien mettre en évidence que c'est un MP envoyé automatiquement.

Karnaj commented 4 years ago

Je considère que c'est un peu comme le MP de prise en charge de validation où l'utilisateur qui a initié l'action est l'auteur du MP.

A-312 commented 4 years ago

C'est bien de pouvoir engager la conversation, ça devrait être la même chose quand un modo résoud une alerte

Karnaj commented 4 years ago

Je mets en Feedback en attendant plus d'avis.

Arnaud-D commented 4 years ago

Je trouve ça bien, mais ça finit par faire vraiment beaucoup de MP. J'ai tellement de MP automatiques que j'ai du mal à trouver mes vrais MP. Je ne suis pas contre en ajouter plus en premier recours, mais je suis preneur de meilleures solutions à plus long terme.

A-312 commented 4 years ago

Je pense qu'on devrait attendre l'ajout des filtres pour les MP : "Sans réponse", "Créé automatiquement"...

Situphen commented 4 years ago

En fait je pense qu'on ne devrait avoir qu'un seul MP "Suivi du contenu X" pour tous les messages automatiques genre ajout d'auteur, suppression d'auteur, réservation, publication, etc.

A-312 commented 4 years ago

Pourquoi on fait un MP et non une notification ?

Situphen commented 4 years ago

Pourquoi on fait un MP et non une notification ?

Pour avoir une trace j'imagine, mais c'est vrai que la question se pose !

Arnaud-D commented 4 years ago

Ces infos-là pourraient être historicisées. C'est un préalable au remplacement des MP par autre chose pour remplir le rôle joué actuellement par les MP automatiques.

A-312 commented 4 years ago

Pourquoi on fait un MP et non une notification ?

Pour avoir une trace j'imagine, mais c'est vrai que la question se pose !

C'est un peu overkill pour garder une trace, ça fait un peu bricolage si c'est la seule raiso. On garde une trace mais on flood pas mal les MP.

J'ai une préférence pour notification + page d'historique dans le tuto Sinon à défaut: Un unique MP avec tous les auteurs

Karnaj commented 4 years ago

Notification + page d'historique des auteurs dans le tuto, ça a l'air plutôt bien. Dans le même genre d'idée, les MPs de suivi de billet pourraient être traités de la même manière (et en fait, les contenus pourraient avoir un historique de publication avec les dates des différentes mises en lignes).

A-312 commented 4 years ago

Je sais que c'est fourre-tout mais ça pourrait faire partie du centre de notification

Par exemple on crée (si ce n'est pas déjà le cas) une table qui log les actions des utilisateurs, on pourrait standardisé cette table : Date Feature Action Element Membre déclencheur Membre cible infos supplémentaires
- tuto add_author contenu/1234 Karnaj A-312 -
Arnaud-D commented 1 year ago

La PR mentionnée ci-dessus ouvre la voie à la solution citée par Karnaj, à savoir "Notification + page d'historique des auteurs dans le tuto". Actuellement, il n'y a pas de notifications encore, mais c'est une évolution naturelle à envisager.