ccnum / ccn

CCN monorepo
0 stars 0 forks source link

Notifications #48

Closed JulMonaco closed 1 month ago

JulMonaco commented 5 months ago

Vérifier que les enseignants reçoivent bien une notification mail lorsqu'une classe publie une réponse. Aujourd'hui ils ont l'air de dire qu'ils ont une notif que si la classe commente ou poste un message dans le forum associé à leur réponse.

Hunerva commented 1 month ago

Vérifier que :

JulMonaco commented 1 month ago

Bonjour @pierretux, Un document qui peut t'aider peut-être : https://docs.google.com/presentation/d/1vTMJAzkU2umtbdWWv0cGRSE4ZRGTpzQrdTC0w0E4vnA/edit#slide=id.g23ddcc20949_0_0

pierretux commented 1 month ago

@Hunerva et @ChristoErasme je viens de mettre à jour la vm de test avec cette refonte, on peut tester ?

ChristoErasme commented 1 month ago

sur https://ccn-dev.thematiques.laclasse.com/ ? j'ai un message d'erreur qd je publie une consigne Image

pierretux commented 1 month ago

@ChristoErasme est maintenant ?

ChristoErasme commented 1 month ago

maintenant la publication de consigne fonctionne, par contre je ne reçois pas de notif sur info@laclasse.com

pierretux commented 1 month ago

Pas encore mais est ce que les profs / intervenant on reçu le mail par contre ?

ChristoErasme commented 1 month ago

Qd j'ai publié la consigne "en tant qu'intervenant", cela n'a pas envoyé de mail Qd j'ai répondu à la consigne "en tant que classe", cela a envoyé un mail à la liste ontourne@lists.erasme.org

Par contre je vois un autre truc : en principe sous la consigne on affiche le nombre de classes sous forme de points gris, qui se colorent lorsque la classe a répondu à la consigne. Là ils n'y sont pas

ChristoErasme commented 1 month ago

fausse alerte, les points gris sous la consigne sont bien affichés sur la CCN en prod

pierretux commented 1 month ago

J'ai changé pour envoyer sur le mail info et non de la liste. Après test en local cela spam tout le monde.

pierretux commented 1 month ago

Des retours ? On passe en prod ?

ChristoErasme commented 1 month ago

pour moi c'est ok mais je n'ai pas la vision de ce que reçoivent les profs, je préfère que ce soit @Hunerva qui fasse le test et te réponde

Hunerva commented 1 month ago

Je viens de faire un test sur ARCHIVES :

A savoir que sur ARCHIVES, le plugin "notifications" n'était pas configuré pour que info@laclasse.com reçoive des notifications. Il faudrait vérifier le paramétrage de ce plugin sur toutes les CCN

ChristoErasme commented 1 month ago

Tu as testé en prod, ce n'est pas plutôt sur ccn-dev.thematiques.laclasse.com qu'il faut le faire @pierretux ?

pierretux commented 1 month ago

Bien sur, on test sur la ccn de dev sinon je ne vois pas à quoi sert un test ....

Hunerva commented 1 month ago

Je tiens à clarifier mon rôle dans l’équipe pour éviter toute confusion. Mon rôle principal est d’assurer le support technique et de remonter les erreurs signalées par les utilisateurs. En ce sens, je ne suis pas directement impliqué dans l'amélioration technique ou dans le suivi des évolutions de la plateforme des CCN.

Je comprends l’importance des tests, mais je pense qu’il serait plus pertinent que ces tâches soient prises en charge par les personnes responsables du produit, comme le Product Owner ou le chef de projet, étant donné que cela dépasse le cadre de mes missions actuelles.

Par ailleurs, je n'étais pas informé de l’existence de cette plateforme de test, et je n'ai pas non plus de compte tests ou accès à celle-ci, ce qui rend difficile pour moi d'effectuer des tests de manière efficace notamment pour les notifications.

Merci de prendre en compte ces éléments et de m’indiquer si d’autres solutions sont envisageables pour la gestion des tests.

pierretux commented 1 month ago

Ok donc de ce que je comprends quand vous étiez avec PA personne testait les dev ? C'était en prod directement ?

Pour moi, dans les projets celui qui remonte les soucis est impliquer dans le tests des soucis remonter avant la mise en prod.

Hunerva commented 1 month ago

Il n'y a jamais eu de processus clair ni de répartition des rôles définie au sein du projet. En ce qui me concerne, tester les évolutions sur la plateforme n'a jamais fait partie de mes responsabilités.

Même si cela avait été le cas, je suis déjà bien occupé par mes tâches quotidiennes liées à laclasse.com, ce qui rend difficile la prise en charge de ce type de mission supplémentaire.

Concernant le projet des CCN, mon rôle se concentre sur la formation et le suivi des utilisateurs, et c'est sur ces aspects que je continue de me concentrer.

pierretux commented 1 month ago

@ChristoErasme Comment on test du coup ?

ChristoErasme commented 1 month ago

Ben pour moi il s'agit de vérifier que le cahier des charges établi par @Hunerva pour les notifs corresponde bien aux besoins et puisse être mis en prod sur toutes les CCN. Je ne vois personne à part lui qui puisse le faire efficacement. L'autre solution consiste à reprendre son tableau et à vérifier que les notifs correspondent... mais il faudrait des comptes fictifs

pierretux commented 1 month ago

C'est en prod, à validé.

Hunerva commented 1 month ago

Depuis la mise en prod, les notifications semblent être envoyées à tous les anciens participants également, ce qui fait de très nombreuses personnes.

Image

ChristoErasme commented 1 month ago

ce collège et cette prof n'étant pas inscrits cette année, cela montre que les notifs sont adressées à tous les comptes (profs) de la CCN, depuis l'origine ? cela fait en effet bps de monde, le mail n'est pas un cas isolé. Mais peut on limiter les notifs à l'année en cours ou faut-il purger les comptes des années précédentes ?

pierretux commented 1 month ago

Pour préciser les infos l'article "Consigne 3" et publier dans l'agenda, ce n'est pas une consigne. IL faut donc que l'on limité les mail à l'année encours.

pierretux commented 1 month ago

@Hunerva Je viens de regarder les notifs et cela fonctionne logiquement en "spammant" les comptes actifs mais je ne peux pas trier suivant les années pour l'agenda. Est ce que tu pourrais délier les auteurs de l'agenda et du blog péda qui ne ont pas de cette année dans chaque ccn ? Comme ça on ne les spammera plus à l'avenir.

Hunerva commented 1 month ago

Ca risque d'être un travail très long mais je vais voir ce que je peux faire ! Je me demande si la solution finalement ne serait pas de "supprimer" tous les utilisateurs puis de réimporter la liste ?

Hunerva commented 1 month ago

Je voulais afficher tous les admins d'une rubrique pour pouvoir ouvrir leur compte et supprimer les affectations mais le bouton "infini" ne semble pas fonctionner.

Window08

Rien que sur "On tourne" ça représente 154 auteurs à enlever à la main. J'aurais jamais le temps et la patience de tout faire pour les 9 CCN.

pierretux commented 1 month ago

Par vague de 10 tu peux le faire du coup

Hunerva commented 1 month ago

Je suis désolé, mais je crains que la tâche demandée ne soit pas réalisable de mon côté dans l'état actuel. Je ne peux pas consacrer des journées entières à supprimer manuellement des comptes de deux rubriques pour les 9 CCN. Le nettoyage de ces comptes n’a jamais été effectué depuis plusieurs années, et je ne pense pas que ce soit à moi seul d’en assumer la responsabilité à ce stade.

Même en supposant que j’aie le temps de m’y consacrer, j'espère que vous comprenez l'ampleur du travail que cela représente. Tout faire manuellement, compte par compte, demanderait un temps considérable.

Je pense qu'il serait préférable de trouver une autre solution ou alors de répartir cette tâche entre toutes les personnes impliquées sur les CCN.