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

Import subjects from pronote #605

Open
wants to merge 24 commits into
base: main
Choose a base branch
from

Conversation

codeuriii
Copy link
Contributor

@codeuriii codeuriii commented Jan 7, 2025

🚀 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

Rajoute un bouton pour récupérer directement les couleurs des matières présentes dans PRONOTE.

@TinAD17tin
Copy link
Contributor

C’est quoi @codeuriii

@codeuriii
Copy link
Contributor Author

J'ai pas fini de dev

@TinAD17tin
Copy link
Contributor

Ah ok j’ai compris j’avais pas regardé le commentaire de PR.
Mais du coup ça veut dire Pawnote le supporte ? (J’en sais absolument rien, juste je m’interroge)

@codeuriii
Copy link
Contributor Author

Bah je part du postulat que si l'emploie du temps peut récupérer les cours, j'imagine que je peux récupérer leurs couleurs avec ce même code

@codeuriii codeuriii marked this pull request as ready for review January 19, 2025 15:04
@codeuriii codeuriii requested a review from tryon-dev as a code owner January 19, 2025 15:04
@codeuriii
Copy link
Contributor Author

Et voila on verra bien ce que ca donne

@raphckrman
Copy link
Contributor

Possible de régler les erreurs de Lint ?
Visible ici: https://github.com/PapillonApp/Papillon/actions/runs/12854670365/job/35839323175?pr=605

@raphckrman
Copy link
Contributor

Et possible d'avoir des screens du rendu aussi ?

@raphckrman
Copy link
Contributor

image
ça ne marche pas

@raphckrman
Copy link
Contributor

Quand tu importes, il faudrait un Spinner qui montre que ça charge peut-être ? Sinon tu as l'impression que juste ça ne fait rien.

Le bouton Pronote n'est pas très intuitif mais ça va encore.
Quand ça sera patch je pourrais test 👍🏼

@codeuriii
Copy link
Contributor Author

l'erreur ne viens pas de mon code, ca marche chez moi. Tu as essayer de te re log de pronote ? Il me semble que j'ai eu cette erreur mais que maintenant ca fonctionne. De plus il faut que tu aille sur plusieurs pages de ton emploie du temps pour que l'expérience soit optimale.

  • Spinner le temps de chargement

@codeuriii
Copy link
Contributor Author

et comment on fait pour résoudre les erreurs lint ? je n'ai pas d'erreur dans mon code

@raphckrman
Copy link
Contributor

l'erreur ne viens pas de mon code, ca marche chez moi. Tu as essayer de te re log de pronote ? Il me semble que j'ai eu cette erreur mais que maintenant ca fonctionne. De plus il faut que tu aille sur plusieurs pages de ton emploie du temps pour que l'expérience soit optimale.

  • Spinner le temps de chargement

Il faut aller sur plusieurs pages que ça fonctionne ? C'est pas bon ça dcp :/

@raphckrman
Copy link
Contributor

et comment on fait pour résoudre les erreurs lint ? je n'ai pas d'erreur dans mon code

Je crois que tu peux faire eslint --fix
Et l'erreur vient de ta PR vu qu'elle arrive quand je clique sur le bouton importer, j'ai tester sur un simulateur sans données donc j'ai connecté mon compte Pronote juste avant de tester. L'important est que ça fonctionne chez tout le monde, si ce n'est pas le cas au moins avoir une erreur qui s'affiche.

@codeuriii
Copy link
Contributor Author

l'erreur ne viens pas de mon code, ca marche chez moi. Tu as essayer de te re log de pronote ? Il me semble que j'ai eu cette erreur mais que maintenant ca fonctionne. De plus il faut que tu aille sur plusieurs pages de ton emploie du temps pour que l'expérience soit optimale.

  • Spinner le temps de chargement

Il faut aller sur plusieurs pages que ça fonctionne ? C'est pas bon ça dcp :/

je sais pas du tout pk ca fais ca, des fois ca fait puis d'un coup ca fais plus et ca revient plus

@codeuriii
Copy link
Contributor Author

le problème c'est que je n'ai plus cette erreur et que je n'ai rien fais de particulier

@codeuriii
Copy link
Contributor Author

j'ai fais le truc de eslint mais il dit que j'ai des import non utilisés mais y a pas

@raphckrman
Copy link
Contributor

j'ai fais le truc de eslint mais il dit que j'ai des import non utilisés mais y a pas

Regarde dans les fichiers, si il dit que y'a des imports pas utilisés c'est que dans ces fichiers y'en a, y'a les fichiers dans l'action si tu veux

@codeuriii
Copy link
Contributor Author

Il me dis surtout que la version de typescript est pas soutenue activement et j'ai regardé tout ce qu'il dit dans settings dev logs c'est utilisé mais soit

@codeuriii
Copy link
Contributor Author

Tu regarderas par toi même vu que tu n'as pas l'air de me croire

@raphckrman
Copy link
Contributor

Il me dis surtout que la version de typescript est pas soutenue activement et j'ai regardé tout ce qu'il dit dans settings dev logs c'est utilisé mais soit

Oui mais ça marche quand même c'est juste un warning je l'ai aussi..

@codeuriii
Copy link
Contributor Author

Tu as regardé le settings dev logs ? Ah bah y a pas d'import qui sont pas utilisés

@codeuriii
Copy link
Contributor Author

faudrait ptet résoudre les problèmes eslint sur la branche principale non ? Je créé la pr sans commit rien du tout et déja des erreurs eslint

@codeuriii
Copy link
Contributor Author

Oe ca marche enfin

@raphckrman
Copy link
Contributor

Voilà, tu as fix l'erreur que j'avais signalé avant ? Que je puisse tester

@codeuriii
Copy link
Contributor Author

Je ne peux rien fais ca ne m'arrive pas (et je dois fix le spinning, j'utilise useState mais ca doit attendre la fin de la fonction donc ca marche pas)

@raphckrman
Copy link
Contributor

Affiche au moins l'erreur ou désactive le bouton si l'utilisateur ne peut pas le faire

@codeuriii
Copy link
Contributor Author

teste quand même j'ai l'impression que le warn ne change rien (pas de spinning)

@codeuriii
Copy link
Contributor Author

Je confirme, le warn n'empeche pas l'execution de mon code

@raphckrman
Copy link
Contributor

Bah si c'est bien le problème, si ya le warn y'a pas l'importation qui se fait c'est ça le problème, d'où l'intérêt d'afficher l'erreur

@codeuriii
Copy link
Contributor Author

Non, le warn ne bloque pas l'importation justement

@raphckrman
Copy link
Contributor

De mon côté, ça change rien justement :/

@codeuriii
Copy link
Contributor Author

Bah je vais regarder parce que je crois que ça vient d'un module externe

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants