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

Fix(Grades): changes + many fix #295

Merged
merged 17 commits into from
Oct 27, 2024
Merged

Conversation

Kgeek33
Copy link
Contributor

@Kgeek33 Kgeek33 commented Oct 18, 2024

🚀 Nouvelle Pull Request

Proposez vos modifications pour améliorer Papillon

Informations importantes

Merci de vous référer à la documentation sur la contribution si vous avez des questions à propos des pull requests (https://gitbook.getpapillon.xyz/organisation/outils-internes/github)

Checklist d'avant pull request

Veuillez cocher toutes les cases applicables en remplaçant [ ] par [x].

  • Vous avez testé de build le projet avec vos modifications et ce build a réussi
  • Vous respectez les conventions de codage et de nommage du projet
  • Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
  • Cette pull request n'est pas un duplicata d'une autre
  • Cette pull request est prête à être revue (review) et fusionnée (merge)
  • Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
  • Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
  • Les détails des changements ont été décrits ci-dessous
  • Cette pull-request n'est pas une "breaking-change" (des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)

Changelogs proposés

  • Correction de l'influence de la note qui, sur la majorité des pages, affichait tout le temps "+ 0,00 pts"
  • Il arrive que Papillon affiche que la pire note/moyenne est de -1 (pour absence, N. Nté...). A la place, ça affichera "??"
  • Pour bien différencier, quand une note a une influence de 0 pts, le texte n'est plus en rouge et affiche à la place du "+" un "+/-"
  • Fix de l'affichage aléatoire du graphe de l'évolution de la moyenne générale
  • Fix de l'affichage non correct de la moyenne générale sur le widget en mode sombre
  • Passage de 3 vibrations à 1 vibration sur le graphe
  • Indentation via Prettier
  • Suppression des import/variables non utilisés dans les pages modifiées

Issues en lien

Issues à closed depuis longtemps

Informations supplémentaires

  • Affichage correct de l'influence + graphe bien affichée
Screenrecorder-2024-10-18-13-41-07-609.mp4
  • Quand influence de la note est de 0

1729251515333

@Louis-htmlcss
Copy link
Contributor

Masterclass

@Louis-htmlcss
Copy link
Contributor

A merge en urgence mdrrr

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 25, 2024

@ecnivtwelve tu peux review stp ?

@yannouuuu yannouuuu added ✨ enhancement New feature or request 🪱 bug Something isn't working labels Oct 26, 2024
@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 26, 2024

Je corrige les conflits sur mes branches et j'arrange ce que dit le lint

@Louis-htmlcss
Copy link
Contributor

mon graphique n'affiche pas le bonnes notes

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 27, 2024

J'ai pas touché au graphique normalement (si juste lors du chargement mais devrait avoir aucune influence)
Je vais tester tout à l'heure pour voir

Copy link
Contributor

@ecnivtwelve ecnivtwelve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ecnivtwelve ecnivtwelve merged commit 051ce13 into PapillonApp:main Oct 27, 2024
1 check passed
@Kgeek33 Kgeek33 deleted the fix/Infosgrades branch October 27, 2024 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment