Update of the Notes Widget on the Papillon home page

[Crazzynel]

🚀 Nouvelle Pull Request

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

Changelog

Improving the GeneralAverageWidget component by dynamically displaying the current period (e.g., "Trimester", "Semester") based on the data from Pronote. Key changes include:

• Dynamically fetched and displayed the "Current Period" under "Mes Notes" in the widget.
• Introduced a new currentPeriod state in useGradesStore to accurately reflect the current period set in Pronote.
• Optimized the fetching logic by ensuring currentPeriod is updated whenever grades or averages are updated.
• Ensured seamless integration without breaking existing functionalities or affecting performance.

These enhancements provide a more accurate and dynamic user experience by correctly reflecting the current grading period from Pronote.

(c.f. picture)

[Gabriel29306]

Vu que ça touche à l'ui, faudrait une review de @ecnivtwelve

Et sinon ça à l'air clean, mais faut que je teste aussi avant de review.

[Crazzynel]

@Gabriel29306

Yup effectivement. Normalement c'est supposé fonctionner correctement. J'ai testé avec 2EDT différents et ça affiche correctement la donnée retournée.

Mais pour l'UI, j'ai voulu renvoyer la donnée de la période affichée dans l'onglet des notes... Il apparaît donc sur la Home page...

[ecnivtwelve]

Pas fan de l'UI, @toi-et-moi ??

[toi-et-moi]

yes pas fan aussi effectivement

[Crazzynel]

Bah... Je vais devoir faire un nouveau commit car j'ai fait une légère erreur... Sans le vouloir j'ai retiré la partie "Moyenne Générale"

Si l'Ui vous plaît pas, jsuis chaud que vous me disiez ce que je devrais améliorer pour le rendre mieux à vos yeux....

Honnêtement, je suis ouvert aux avis et ça me dérange pas d'avoir des retours positifs comme négatifs (ça peut m'aider a m'améliorer tfaçon)
@ecnivtwelve @toi-et-moi

[Louis454545]

Je suis désolée, mais je comprends vraiment pas l'intérêt d'afficher la période

[Crazzynel]

@Louis-htmlcss

Je peux comprendre que l'intérét ne soit pas super clair, mais c'est pour faire paraître globalement sans aller sur l'onglet "Notes".
En gros, l'objectif est que l'utilisateur voie directement la période de notation. Donc, sa moyenne est comprise par exemple, dans le trimestre 1 ou bien le trimestre 2.
Donc une forme de... suivi de la progression. C'est toujours bien de suivre son évolution trimestrielle quand c'est clairement affiché...

En affichant ça directement sur la page d'accueil, on montre aussi le contexte immédiat de notation jpense.

Voilà... si ça peut aider.
Et je peux entendre la remarque du, oui la page des notes fait ça, mais je trouve que c'est pas mal de l'afficher aussi sur la page d'accueil...
C'est comme ça que je le voyais du moins

[Crazzynel]

Voilà !

J'ai corrigé un bug d'affichage.
Si il faut revoir l'UI n'hésitez pas à me le dire !

[Louis454545]

Voilà !

J'ai corrigé un bug d'affichage. Si il faut revoir l'UI n'hésitez pas à me le dire !

Tu peux envoyer un screen de ce que ça fait ?

[Crazzynel]

Ouaip np !

Je finis le build de l'app pour retester, et je fais ça !

[Louis454545]

Ouaip np !

Je finis le build de l'app pour retester, et je fais ça !

sinon tu peux juste utiliser expo go

[Crazzynel]

Ouais ! C'est ce que je fais! Je finis ! et j'envoie ça

[Crazzynel]

@Louis-htmlcss

[Louis454545]

@Louis-htmlcss

Désolé, Je ne suis vraiment pas fan de design

[tom-theret]

Il y a un Figma au cas où, tu y trouvera les widgets à faire.
https://www.figma.com/community/file/1409566366286902640/papillon-interface-utilisateur

[Crazzynel]

Il y a un Figma au cas où, tu y trouvera les widgets à faire. https://www.figma.com/community/file/1409566366286902640/papillon-interface-utilisateur

Ouais jvais faire ça...
Je close la pull request ducoup...

Désolé de la perte de temps... et encore merci à vous tous !!!

[ecnivtwelve]

C'était pas non plus bête comme idée, faudrait juste voir avec @toi-et-moi pour l'interface