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

[Karnaj]

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.

• L'auteur du MP ne doit plus être bot mais l'utilisateur qui a envoyé la requête (self.request.user).
• Il ne faut pas qu'il soit envoyé en tant que staff, donc on supprime l'argument hat de l'envoi du MP.
• On ne veut pas que l'auteur qui a ajouté soit supprimé du MP, donc on met l'argument leave à False (voir send_mp et ses arguments).
• On ne veut pas envoyer de MP si le membre s'ajoute lui-même (un membre staff par exemple), donc on ne l'envoie que si l'utilisateur est différent de celui qui a envoyé la requête.

[Situphen]

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]

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]

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

[Karnaj]

Je mets en Feedback en attendant plus d'avis.

[Arnaud-D]

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]

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

[Situphen]

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]

Pourquoi on fait un MP et non une notification ?

[Situphen]

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]

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]

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]

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]

~~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]

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.