Skip to content
This repository has been archived by the owner on Aug 25, 2024. It is now read-only.

Ajout d'un bouton pour afficher le mot de passe #42

Merged
merged 2 commits into from
Mar 7, 2024
Merged

Ajout d'un bouton pour afficher le mot de passe #42

merged 2 commits into from
Mar 7, 2024

Conversation

Sacha338
Copy link
Contributor

@Sacha338 Sacha338 commented Mar 7, 2024

Checklist d'avant pull request

  • Vous avez testé de build le projet avec vos modifications et ce build a réussi
  • Vous respectez les conventions de codage et de nomage 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)
  • Cette pull request doit être fusionnée dans la branche development (le cas contraire préciser laquelle)
  • 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écris ci-dessous
  • Cette pull-request n'est pas une "breaking-change" (par exemple des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)

Explications des changements

Changelogs proposés

[Ajout d'un bouton permettant d'afficher et de masquer le mot de passe sur l'écran de connexion Pronote]

Note

Pour que votre pull-request soit fusionnée, vous devez obtenir l'approbation de au moins deux membres de l'équipe dont un coordinateur.
Soyez donc patient :)

Informations supplémentaires

[Insérez ici vos informations supplémentaires si besoin]

Copy link
Member

@Vexcited Vexcited left a comment

Choose a reason for hiding this comment

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

LGTM, faut juste régler quelques soucis concernant sur le fetching de la session et la gestion d'erreur.

views/NewAuthStack/Pronote/NGPronoteLogin.tsx Outdated Show resolved Hide resolved
views/NewAuthStack/Pronote/NGPronoteLogin.tsx Outdated Show resolved Hide resolved
views/NewAuthStack/Pronote/NGPronoteLogin.tsx Outdated Show resolved Hide resolved
views/NewAuthStack/Pronote/NGPronoteLogin.tsx Show resolved Hide resolved
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.

Nickel !

@LeGeek01 LeGeek01 requested a review from Vexcited March 7, 2024 16:43
@ecnivtwelve ecnivtwelve merged commit 9cc7fdf into PapillonApp:development Mar 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants