Oh mrd, 3 reviewer pour codeowner, bon bah je reviens plus tard

@Kgeek33 tu pourrais regarder ?

@Kgeek33 tu pourrais regarder ?

ok je regarde ça de suite, ça a l'air intéressant comme pr en lisant la description :)

ok je fais une review de code après mais je remarque déjà des problèmes depuis la maj de Pawnote sur la 7.7.0 :

1. Quand aucune note prise en compte sur une matière
   
   

2. Les -1/20 prient en compte ! Faudrait genre remplacer par NaN ou supprimer à certains endroits :
   
   Du coup, l'impact de la moyenne sur une matière est complètement fausse (pas besoin de capture)

@Kgeek33 c'est corrigé

J'ai pas corrigé pour les imports, ça peut se faire dans une pr à part

Je review plus tard

ah je viens de comprendre pourquoi l'influence de la moyenne est complètement fausse…

ça filtre qu'une note et non toutes les notes d'une matière 🤦‍♂️
https://github.com/Gabriel29306/PapillonV7/blob/75f455f69f1c91de0aa603f3b11e093f672189f1/src/utils/grades/getAverages.ts#L167

mais j'suis en train de review et j'vais te proposer un fichier bien clean

À chaque résolution de conflits, je me mange des nouvelles erreurs TS, youpi

Je vois pas comment tu peux avoir cette erreur, vraiment. Tu as vidé ton cache de build (normlement il n'y a pas besoin, mais vu ton cas...) ?

npm ci + npx expo start -c et déco / reco du compte, et toujours le même message...

npm ci + npx expo start -c et déco / reco du compte, et toujours le même message...

Faudrait que tu ouvres une issue du côté de Literate

@Gabriel29306
Toutes les personnes qui ont signalé des problèmes après le problème de Pawnote rencontrent la même erreur que @Bulgus et ça c'est depuis la maj de Pronote

C'est pas nouveau mais oui, il faut ouvrir une issue depuis Pawnote et non Papillon (pour le moment :))

@Gabriel29306 @Kgeek33
Okay je vais aller voir de leur côté !

@Gabriel29306
Toutes les personnes qui ont signalé des problèmes après le problème de Pawnote rencontrent la même erreur que @Bulgus et ça c'est depuis la maj de Pronote

C'est pas nouveau mais oui, il faut ouvrir une issue depuis Pawnote et non Papillon (pour le moment :))

Mais chez moi ça marche, j'ai fait la PR chez Pawnote et elle a été merge

@Gabriel29306
Toutes les personnes qui ont signalé des problèmes après le problème de Pawnote rencontrent la même erreur que @Bulgus et ça c'est depuis la maj de Pronote

C'est pas nouveau mais oui, il faut ouvrir une issue depuis Pawnote et non Papillon (pour le moment :))

Mais chez moi ça marche, j'ai fait la PR chez Pawnote et elle a été merge

Oui ça marche de mon côté également mais sur certains établissements, il doit y avoir une subtilité qui fait que cette erreur survient
Ça peut être les moyennes générales désactivées tout comme d'autres choses...

Oui ça marche de mon côté également mais sur certains établissements, il doit y avoir une subtilité qui fait que cette erreur survient
Ça peut être les moyennes générales désactivées tout comme d'autres choses...

Ils font suer, mais on continuera à s'adapter, pas le choix

@Gabriel29306
Patch dans la dernière version de Pawnote (1.4.1) possible de faire la MAJ sur cette PR ?

Faudrait que j'intègre ncu pour mettre à jour vers les dernières dépendances (sans toucher à expo 51 => 52)

Nouveau problème avec la dernière MAJ de Pawnote : la moyenne calculée est de NaN et cause un crash

C'est bon, j'ai mis à jour Pawnote ici

Et je me suis (encore) occupé du lint

Je vais m'énerver sur un fucking merge car j'ai dû faire un minimum de lint ????

Fait suer