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

feat(Pronote): Popup informative #297

Merged
merged 13 commits into from
Oct 27, 2024
Merged

Conversation

Kgeek33
Copy link
Contributor

@Kgeek33 Kgeek33 commented Oct 19, 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

  • Ajout d'une popup pour dire que les comptes autres que les élèves ne sont pas compatibles
  • Optimisation du code

Closed #155

Informations supplémentaires

Screenrecorder-2024-10-19-21-27-38-551.mp4

1729366075946

@Kgeek33 Kgeek33 requested a review from Gabriel29306 October 21, 2024 17:25
@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 21, 2024

ah oui, j'avais activé les tabulations sur Prettier, voilà pq ça a du faire ça
Désactivé et corrigé normalement

Copy link
Contributor

@Gabriel29306 Gabriel29306 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

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.

Un peu intrusif nan ?

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 25, 2024

Jspas. Pour l'instant, quand un parent se connecte, il y a un chargement infini
À toi de me dire et je corrige vite fait ;)

@ecnivtwelve
Copy link
Contributor

Jspas. Pour l'instant, quand un parent se connecte, il y a un chargement infini À toi de me dire et je corrige vite fait ;)

Par !url.includes("identifiant"), il check si ça load en boucle c'est ça ?

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 26, 2024

Quand la personne se connecte à Pronote, Pawnote force la redirection vers un compte élève (donc url : mobile.eleve.html)
Quand c'est un compte élève, l'URL contient identifiant=...

S'il n'y a pas identifiant=, ça veut dire que tu essayes de te connecter à un compte élève alors que tes identifiants sont conçus pour un autre compte

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 26, 2024

En version simplifié, ça boucle à l'infini en ce moment car Pawnote ne trouve pas dans l'URL identifiant=
identifiant= permet de faire le token si je comprends bien la logique

@yannouuuu yannouuuu added ✨ enhancement New feature or request 🔌 ENT / services This issue is linked with Pawnote or a school/external service 🎨 interface Element visible to the user labels Oct 26, 2024
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.

Ça me paraît bien !

@ecnivtwelve ecnivtwelve merged commit d76b5cd into PapillonApp:main Oct 27, 2024
1 check passed
@Kgeek33 Kgeek33 deleted the feat/msgParents branch October 27, 2024 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement New feature or request 🔌 ENT / services This issue is linked with Pawnote or a school/external service 🎨 interface Element visible to the user
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Accès Parent vide
5 participants