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: Add automatic login for demo instance in PronoteCredentials #89

Closed
wants to merge 1 commit into from

Conversation

Louis-htmlcss
Copy link
Contributor

🚀 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 de la connexion automatique pour l'instance de démonstration Pronote

Description

Cette PR ajoute une fonctionnalité permettant la connexion automatique lorsqu'une instance de démonstration Pronote est détectée. Elle améliore l'expérience utilisateur en simplifiant le processus de connexion pour les comptes de démonstration.

Changements principaux

  • Modification de PronoteCredentials.tsx pour gérer les identifiants de démonstration
  • Mise à jour de PronoteManualURL.tsx pour rediriger vers PronoteCredentials avec les identifiants pré-remplis
  • Ajout d'un useEffect dans PronoteCredentials pour déclencher la connexion automatique

Détails techniques

  • Utilisation de useState pour initialiser les champs username et password avec les valeurs de démonstration
  • Implémentation d'un useEffect pour soumettre automatiquement le formulaire si des identifiants de démonstration sont fournis
  • Modification de la fonction handleLogin pour accepter les paramètres username et password

Tests effectués

  • Vérification du fonctionnement correct de la connexion automatique pour l'instance de démonstration
  • Test du comportement normal pour les instances non-démo

Notes supplémentaires

Cette fonctionnalité facilite les tests et les démonstrations de l'application, tout en maintenant l'avertissement pour les utilisateurs concernant la nature potentiellement instable des instances de démonstration.

@Gabriel29306
Copy link
Contributor

C'est une blague hein ?

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.

2 participants