[Feature]: Les groupes ou classes ne s'affichent pas toujours dans l'emploi du temps

[Lukiluc29]

Description de la feature

Quand je clique sur un cours en groupe, celui la m'affiche correctement le/les groupe(s) qui y ont assistés. Cependant, quand je clique sur un cours en classe entière il n'affiche pas la/les classe(s) qui y assisteront ce qui est embetant pour savoir avec qui on sera.

Comportement attendu

Afficher tout le temps les groupes ou classes qui assisteront au cours

[Kgeek33]

ce n'est pas un comportement normal ?
ça affiche les groupes/classes uniquement si le cours n'est pas en classe entière

[Lukiluc29]

ce n'est pas un comportement normal ? ça affiche les groupes/classes uniquement si le cours n'est pas en classe entière

Ne serait-il pas judicieux de ne pas afficher tout le temps la classe car par exemple j'ai un cours exceptionnel vendredi avec deux autres classes et il n'affiche pas les classes qui s'y trouveront aussi, alors que sur pronote c'est bien ecris les autres classes de ce cours

[Kgeek33]

ce n'est pas un comportement normal ? ça affiche les groupes/classes uniquement si le cours n'est pas en classe entière

Ne serait-il pas judicieux de ne pas afficher tout le temps la classe car par exemple j'ai un cours exceptionnel vendredi avec deux autres classes et il n'affiche pas les classes qui s'y trouveront aussi, alors que sur pronote c'est bien ecris les autres classes de ce cours

ta suggestion est intéressante, à voir si l'intégration est possible
du coup, ce n'est plus un bug mais une feature :)

[Lukiluc29]

ce n'est pas un comportement normal ? ça affiche les groupes/classes uniquement si le cours n'est pas en classe entière

Ne serait-il pas judicieux de ne pas afficher tout le temps la classe car par exemple j'ai un cours exceptionnel vendredi avec deux autres classes et il n'affiche pas les classes qui s'y trouveront aussi, alors que sur pronote c'est bien ecris les autres classes de ce cours

ta suggestion est intéressante, à voir si l'intégration est possible du coup, ce n'est plus un bug mais une feature :)

Oui, finalement le bug a tourné à la feature mdr

[Gabriel29306]

Question:
Si je m'intéresse à la feature, faudrait-il l'afficher directement ou uniquement dans les détails du cours ?
(Mentionnez moi pour que je puisse voir la réponse)

[raphckrman]

@Gabriel29306 Dans les détails du cours je pense, en tout cas il me semble que c'était déjà fait comme ça avant

[Gabriel29306]

Oki

[Gabriel29306]

En fait, je viens de regarder, ils sont déjà là.
Le seul truc que je peux faire c'est ajouter ça directement sur la page de cours

[Kgeek33]

En fait, je viens de regarder, ils sont déjà là.
Le seul truc que je peux faire c'est ajouter ça directement sur la page de cours

Oui c'est déjà intégré. Il faut juste le rendre tout le temps visible
Faut changer ici => https://github.com/PapillonApp/Papillon/blob/main/src%2Fviews%2Faccount%2FLessons%2FDocument.tsx#L174-L179

[Gabriel29306]

Je viens de regarder la condition en question:

{
  icon: <Users />,
  text: lesson.group?.includes(",") ? "Groupes" : "Groupe",
  value: lesson.group?.replace(/\[|\]/g, ""),
  enabled: Boolean(lesson.group?.trim()), // Check if lesson.group exists, is not null, and not empty
},

La seule condition à l'air d'être une valeur existante

[Kgeek33]

Je viens de regarder la condition en question:

{
  icon: <Users />,
  text: lesson.group?.includes(",") ? "Groupes" : "Groupe",
  value: lesson.group?.replace(/\[|\]/g, ""),
  enabled: Boolean(lesson.group?.trim()), // Check if lesson.group exists, is not null, and not empty
},

La seule condition à l'air d'être une valeur existant

Ah donc eft l'intégration n'a pas l'air possible, j'pense que lesson.group est undefined si c'est un cours en classe entière
J'vais aller vérifier et j'vous dis ça

[Gabriel29306]

En classe entière, c'est undefined
On peut éventuellement afficher aussi la classe

[Kgeek33]

En classe entière, c'est undefined
On peut éventuellement afficher aussi la classe

Après, ça semble logique puisque lesson.group affiche les groupes et non les classes
Oui idée intéressante mais pour répondre à la feature, essayer de voir avec Pawnote pour voir si sur un cours, ça renvoie les classes et non (ou en plus) les groupes

[Kgeek33]

@Gabriel29306 je devrai faire une issue à part mais je demande quand même, c'est dans le rapport : il serait intéressant d'intégrer comme sur Pronote la section "Notes" sur un cours
Il faut juste afficher lesson.additionalNotes