Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notes en double en cas de même matière #268

Closed
LeGeek01 opened this issue Oct 6, 2024 · 16 comments
Closed

Notes en double en cas de même matière #268

LeGeek01 opened this issue Oct 6, 2024 · 16 comments
Labels
🪱 bug Something isn't working ✖️ invalid This doesn't seem right

Comments

@LeGeek01
Copy link
Contributor

LeGeek01 commented Oct 6, 2024

Description du bug

Pour nous les STI2D, les maths spé et la physique sont regroupées dans une même matière pour les notes : "Maths-physique-chimie". Par contre, il y a deux fois la matière, une pour les maths spé et une pour la physique.
Pronote fait bien la différence au niveau des notes, mais Papillon duplique les notes d'une catégorie tout en prennant la bonne moyenne.

Étapes à reproduire

  1. Tu veux vraiment que je t'expliques comment arriver en sti2d ???

Comportement attendu

Que les notes soient différenciées

Appareil

Samsung Galaxy A13

Version du système d`exploitation

Android 14

Version

7.2.1-beta

Environnement

Application native Android

Source de l'application

Stores (Play Store / App Store)

Service scolaire

🦋 Pronote

Captures d'écran / vidéo

IMG-20241006-WA0000

Contexte supplémentaire

No response

@Cleboost
Copy link
Contributor

Cleboost commented Oct 7, 2024

Up 👍

Screenshot_20241006_232406_PRONOTE Screenshot_20241006_232338_Papillon

@yannouuuu yannouuuu added ✖️ invalid This doesn't seem right 🪱 bug Something isn't working labels Oct 7, 2024
@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 8, 2024

@Vexcited
Alors j'ai essayé de résoudre cette issue mais je me suis rendu compte qu'en fait, c'était un bug de Pawnote.

  • Bug : Quand Pawnote va récupérer les moyennes de chaque matière, si 2 matières a le même nom, alors une des 2 renverra NaN
  • Suggestion : Et pour faciliter la différenciation entre les même noms de matière, est-ce que ça serait possible d'ajouter un argument sur les notes (par exemple, quel prof a saisi la note) pour éviter les regroupements des notes dans les mauvaises matières ?

@Gabriel29306
Copy link
Contributor

Je n'ai pas de problème où il y a une moyenne en NaN.
La technique d'utiliser le professeur n'est, d'après moi, car un prof peut être utilisé pour 2 matières de même nom

@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 9, 2024

Oui bien sûr mais en tout cas, ajouter un argument pour bien différencier les matières

@Gabriel29306
Copy link
Contributor

@Vexcited les matières ont normalement des Ids ?

@Vexcited
Copy link
Member

Vexcited commented Nov 9, 2024

oui y a des id mais ils sont dynamiques en fonction de ta session

@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 9, 2024

du coup, on peut pas différencier les même matières ?

@Vexcited
Copy link
Member

Vexcited commented Nov 9, 2024

entre deux sessions, non on peut pas, si c'est sur la même session cependant, oui on peut

@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 9, 2024

ah oui en fait, je me souviens que pour différencier les matières, c'est possible mais différencier les notes appartenant à une matière de même nom, c'est pas possible

@Kgeek33
Copy link
Contributor

Kgeek33 commented Nov 9, 2024

Pask l'id de la note, c'est le nom de la matière + timestamp de quand a été crée la note

@Kgeek33
Copy link
Contributor

Kgeek33 commented Dec 12, 2024

L'issue peut être close, corrigé dans #404 !

@Kgeek33
Copy link
Contributor

Kgeek33 commented Dec 14, 2024

@LeGeek01 sur la version 7.6.0 de Papillon, rencontres-tu encore ce problème ?
Perso je rencontre plus ce problème

@LeGeek01
Copy link
Contributor Author

firefox n'a pas trouvé l'issue, j'ai une erreur #404 ... (blague de merde j'accepte de sortir)
la 7.6.0 n'a pas encore été acceptée sur le play store donc je saurais pas te dire, il faudrait que je lance avec expo go pour confirmer

@Kgeek33
Copy link
Contributor

Kgeek33 commented Dec 14, 2024

firefox n'a pas trouvé l'issue, j'ai une erreur #404 ... (blague de merde j'accepte de sortir)
la 7.6.0 n'a pas encore été acceptée sur le play store donc je saurais pas te dire, il faudrait que je lance avec expo go pour confirmer

J'aime bien la blague 😂
C'est bizarre, ça m'a proposé la version 7.6.0 sur le play store à midi, peut-être qu'elle a été déployé à peu de personnes

@LeGeek01
Copy link
Contributor Author

Je viens de recevoir la mise à jour (5 jours après quand même !!) et le bug est fixé

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪱 bug Something isn't working ✖️ invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

6 participants