Issue 2 minor bug

[Hamza-Falih]

No description provided.

[dcadiou]

Salut Mohamed. J'ai accepté ta pull resuest. Je me suis rendu compte au passage que beaucoup de test du processus d'intégration continue échouent (voir [ https://github.com/grp-attestoodle/moodle-block_attestoodle/actions | https://github.com/grp-attestoodle/moodle-block_attestoodle/actions ] ). Ceci dit ce n'est pas lié à ton intervention. Visiblement le problème existait bien avant. Du coup j'ai un peu travaillé dessus, et ai créé à mon tour une pull request qui fait passer tous les tests en question (voir [ #6 | #6 ] ). Pourrais-tu à ton tour la valider ? A Caen nous n'utilisons pas ce plugin block. D'ailleurs je ne sais pas ce qu'il fait. Marc ne nous présentait aucun des nouveaux sous-plugin qu'il ajoutait. Du coup je ne peux pas tester si ce que j'ai changé à un impact néfaste ou pas. J'ai juste observé que c'est efficace sur les erreurs des tests CI et je me suis arrêté là. Pour info, j'ai documenté les problèmes détectés par les tests CI et pourquoi je les ai résolu comme ça. Voir [ #5 | #5 ] Bon week end Didier CADIOU développeur informatique • applications & technologies centre d'enseignement multimedia universitaire Campus 1, bât B, esplanade de la paix - CS 14032 14032 Caen cedex 05 unicaen · cemu · 02 31 56 67 38 De: "Mohamed Hamza Falih" ***@***.***> À: "grp-attestoodle/moodle-block_attestoodle" ***@***.***> Cc: "Didier Cadiou" ***@***.***>, "Review requested" ***@***.***> Envoyé: Jeudi 24 Novembre 2022 16:14:22 Objet: Re: [grp-attestoodle/moodle-block_attestoodle] Issue 2 minor bug (PR #4) [ https://github.com/Hamza-Falih | @Hamza-Falih ] requested your review on: [ #4 | #4 ] Issue 2 minor bug. — Reply to this email directly, [ #4 (comment) | view it on GitHub ] , or [ https://github.com/notifications/unsubscribe-auth/AGD64EBGWWOJY535U5CCHPLWJ6A45ANCNFSM6AAAAAASKQDO5Y | unsubscribe ] . You are receiving this because your review was requested. Message ID: <grp-attestoodle/moodle-block_attestoodle/pull/4/issue_event/7886926195 @ github . com>

[Hamza-Falih]

Salut @dcadiou,

Merci beaucoup d'avoir accepté ma Pull Request. En effet, j'avais ajouté les tests d'intégration continu à ce plugin pour respecter la procédure de développement Attesdoodle et j'avais remarqué que les résultats n'était pas concluant aussi. Oui, ce n'était pas lié à mon intervention puisqu'ils échouaient au niveau de la branche master bien avant de valider la Pull Request.

Merci aussi d'avoir travaillé sur ces tests et d'avoir corrigé les problèmes qui étaient dessus. J'essaierai de lire ce que tu as fait pour valider la Pull Request. En fait, ce plugin permet d'ajouter un block Attestoodle à un cours qui affiche le nombre de jalons et formations.
Ci-après une capture :

Merci pour la documentation que tu as fait aussi, j'esserai d'y jeter un coup d'oeil.

Bon après-midi,

[dcadiou]

Salut Mohamed. Merci pour les informations. Concernant ce bloc, je suis étonné d'y trouver des informations qui sont extérieures au cours et ne le concernent pas non plus nécessairement (nombre de formations, liste des formations par exemple). Conceptuellement, je trouve que cela pose question. En revanche, l'idée d'avoir un bloc qui donne des informations Attestoodle qui concernent le cours courant (sur les jalons par exemple), pourquoi pas. Quoi qu'il en soit, le plugin existe et est comme ça. Je ne sais pas où il est utilisé, ni même s'il est beaucoup utilisé. Il existe bien d'autres priorités sur le projet Attestoodle. Bon courage. Didier CADIOU développeur informatique • applications & technologies centre d'enseignement multimedia universitaire Campus 1, bât B, esplanade de la paix - CS 14032 14032 Caen cedex 05 unicaen · cemu · 02 31 56 67 38 De: "Mohamed Hamza Falih" ***@***.***> À: "grp-attestoodle/moodle-block_attestoodle" ***@***.***> Cc: "Didier Cadiou" ***@***.***>, "Mention" ***@***.***> Envoyé: Lundi 28 Novembre 2022 15:49:10 Objet: Re: [grp-attestoodle/moodle-block_attestoodle] Issue 2 minor bug (PR #4) Salut [ https://github.com/dcadiou | @dcadiou ] , Merci beaucoup d'avoir accepté ma Pull Request. En effet, j'avais ajouté les tests d'intégration continu à ce plugin pour respecter la procédure de développement Attesdoodle et j'avais remarqué que les résultats n'était pas concluant aussi. Oui, ce n'était pas lié à mon intervention puisqu'ils échouaient au niveau de la branche master bien avant de valider la Pull Request. Merci aussi d'avoir travaillé sur ces tests et d'avoir corrigé les problèmes qui étaient dessus. J'essaierai de lire ce que tu as fait pour valider la Pull Request. En fait, ce plugin permet d'ajouter un block Attestoodle à un cours qui affiche le nombre de jalons et formations. Ci-après une capture : [ https://user-images.githubusercontent.com/34319005/204306836-2fff29e9-9210-4093-a98d-2711609a2113.png ] Merci pour la documentation que tu as fait aussi, j'esserai d'y jeter un coup d'oeil. Bon après-midi, — Reply to this email directly, [ #4 (comment) | view it on GitHub ] , or [ https://github.com/notifications/unsubscribe-auth/AGD64ECBQVYYO2KONMUXP6LWKTA6NANCNFSM6AAAAAASKQDO5Y | unsubscribe ] . You are receiving this because you were mentioned. Message ID: <grp-attestoodle/moodle-block_attestoodle/pull/4/c1329235150 @ github . com>

[Hamza-Falih]

Salut @dcadiou,

De rien. C'est ce que je sais en tout cas.

Oui, je suis d'accord avec toi que les informations sur le nombre de formations et leurs noms ne devraient pas être affichés dans un cours.

Je ne sais pas non plus s'il est utilisé et par qui. Je ne l'ai découvert que récemment.

Merci ! Bon courage à toi aussi !

Bon après-midi,

[dcadiou]

Salut Mohamed. Juste un élément d'information supplémentaire, une collègue me faisait remarquer qu'un bloc peut être ajouté au niveau d'une catégorie de cours. Dans ce contexte, il devient plus pertinent d'y lister les formations. Donc mon commentaire précédent n'est peut-être pas valide. Didier CADIOU développeur informatique • applications & technologies centre d'enseignement multimedia universitaire Campus 1, bât B, esplanade de la paix - CS 14032 14032 Caen cedex 05 unicaen · cemu · 02 31 56 67 38 De: "Mohamed Hamza Falih" ***@***.***> À: "grp-attestoodle/moodle-block_attestoodle" ***@***.***> Cc: "Didier Cadiou" ***@***.***>, "Mention" ***@***.***> Envoyé: Mercredi 30 Novembre 2022 14:37:52 Objet: Re: [grp-attestoodle/moodle-block_attestoodle] Issue 2 minor bug (PR #4) Salut [ https://github.com/dcadiou | @dcadiou ] , De rien. C'est ce que je sais en tout cas. Oui, je suis d'accord avec toi que les informations sur le nombres de formations et leurs noms ne devraient pas être affichés dans un cours. Je ne sais pas non plus s'il est utilisé et par qui. Je ne l'ai découvert que récemment. Merci ! Bon courage à toi aussi ! Bon après-midi, — Reply to this email directly, [ #4 (comment) | view it on GitHub ] , or [ https://github.com/notifications/unsubscribe-auth/AGD64EEZG63EGCNEOVVN74LWK5KDBANCNFSM6AAAAAASKQDO5Y | unsubscribe ] . You are receiving this because you were mentioned. Message ID: <grp-attestoodle/moodle-block_attestoodle/pull/4/c1332164501 @ github . com>

[Hamza-Falih]

Bonjour @dcadiou,

Merci pour l'information. En effet, s'il est possible d'ajouter un bloc au niveau d'une catégorie de cours, l'affichage des formations devient plus pertinent dans ce cas.

Bon après-midi,