[MOT DE PASSE OUBLIE] Saisis l'email puis modifie le mot de passe #737
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
firefly33
changed the title
firefly33
force-pushed
the
pages-reinitialisation-mot-de-passe
branch
2 times, most recently
from
18b7255
to
c7aa506
Compare
firefly33
force-pushed
the
pages-reinitialisation-mot-de-passe
branch
from
c7aa506
to
9840a7e
Compare
firefly33
changed the title
bbougon
reviewed
Nov 14, 2024
| @@ -46,6 +48,14 @@ export const RouteurPublic = () => { | |||
| <Route path="connexion" element={<EcranConnexion />} /> | |||
| </Route> | |||
|
|
|||
| <Route path="/utilisateur" element={<LayoutPublic />}> | |||
There was a problem hiding this comment.
Passer la prop pour ne pas afficher le menu navigation publique
bbougon
reviewed
Nov 14, 2024
| <main role="main" className="ecran-mot-de-passe-oublie"> | ||
| <div className="formulaire-colonne-gauche"> | ||
| <div className="fr-container"> | ||
| <TypographieH3 style={{ textAlign: 'center' }}> |
bbougon
reviewed
Nov 14, 2024
| </div> | ||
| </div> | ||
| <div className="fond-clair-mac icone-colonne-droite"> | ||
| <img src={illustrationSecuritePostesSvg} alt="" /> |
bbougon
reviewed
Nov 14, 2024
| <i className="mac-icone-information" /> | ||
| <div> | ||
| Si le compte existe, un e-mail de redéfinition de mot de passe sera | ||
| envoyé à : <b>[email protected]</b> Veuillez vérifier votre boîte de |
bbougon
reviewed
Nov 14, 2024
| </div> | ||
| ); | ||
|
|
||
| return <FormulaireMotDePasseOublie surSoumission={mutate} />; |
There was a problem hiding this comment.
On a besoin du mail (cf commentaire précédent), voir :
- si react query permet d’obtenir la valeur passée
- sinon, appeler une callback intermédiaire pour conserver le mail puis appeler le mutate
bbougon
reviewed
Nov 14, 2024
|
|
||
| return ( | ||
| <div className="formulaire-mot-de-passe-oublie-layout"> | ||
| <p style={{ textAlign: 'center' }}> |
bbougon
reviewed
Nov 14, 2024
| adresseElectroniqueSaisie(e.target.value) | ||
| ) | ||
| } | ||
| onChange={(e) => |
There was a problem hiding this comment.
Peut-être que le onChange ne devrait pas afficher l’erreur tant que le mail n’est pas valide. Il pourrait se focaliser uniquement sur l’état du bouton Réinitialiser le mot de passe. Le onBlur servant à la validation
bbougon
reviewed
Nov 14, 2024
| <Button | ||
| type="button" | ||
| variant="secondary" | ||
| onClick={() => window.location.replace('/')} |
There was a problem hiding this comment.
Passer par le navigate pour éviter le rechargement de la page et ainsi éviter un appel à l’API.
Faire attention aux liens hateoas (sont-ils tous présents => se-connecter)
bbougon
reviewed
Nov 14, 2024
| ...construisErreurChamp(estChampValide), | ||
| }, | ||
| }), | ||
| ...Object.entries(ajouteAuNouvelEtat()).reduce((prev, [clef, valeur]) => { |
There was a problem hiding this comment.
renommer prev en precedent
bbougon
reviewed
Nov 14, 2024
| <main role="main" className="ecran-reinitialiser-mot-de-passe"> | ||
| <div className="formulaire-colonne-gauche"> | ||
| <div className="fr-container"> | ||
| <TypographieH3 style={{ textAlign: 'center' }}> |
bbougon
reviewed
Nov 14, 2024
| <TypographieH3 style={{ textAlign: 'center' }}> | ||
| Réinitialisation de votre mot de passe | ||
| </TypographieH3> | ||
| <FormulaireReinitialiserMotDePasseConnecte |
There was a problem hiding this comment.
Le suffixe Connecte n’est pas très parlant, voir si on trouve un meilleur nom pour désigner l’intention qui est de faire l’appel au back
bbougon
reviewed
Nov 14, 2024
|
|
||
| return ( | ||
| <div className="formulaire-reinitialiser-mot-de-passe-layout"> | ||
| <p style={{ textAlign: 'center' }}> |
bbougon
reviewed
Nov 14, 2024
| <li>1 majuscule</li> | ||
| <li>1 minuscule</li> | ||
| <li>1 chiffre</li> | ||
| <li>1 caractère spécial parmi #?!@$%^&*-'+_()[]</li> |
There was a problem hiding this comment.
remplacer #?!@$%^&*-'+_()[] par #?!@$%^&*-'+_()[]
bbougon
reviewed
Nov 14, 2024
| motDePasse: string; | ||
| confirmationMotDePasse: string; | ||
| }) => { | ||
| if (!token || token.length === 0 || token === '') |
There was a problem hiding this comment.
faire token.trim() === ''
bbougon
reviewed
Nov 14, 2024
| ); | ||
| }, | ||
| onSuccess: () => { | ||
| navigate('/aidant/tableau-de-bord'); |
There was a problem hiding this comment.
L’utilisateur n’est pas connecté => renvoyer vers l’écran de connexion
firefly33
force-pushed
the
pages-reinitialisation-mot-de-passe
branch
from
62432ec
to
1cdc254
Compare
…sie de l'email [MOT DE PASSE OUBLIE] Créé le réducteur pour le formulaire de saisie de l'email
… case pour gérer les types de toasts
…ie de l'email sur l'action HATEOAS [WIP][REINITIALISER MOT DE PASSE] dev en cours
…lise a pour afficher le lien vers la page de réinitialisation
…gistrer le nouveau mot de passe
…tion dans le dossier domaine/authentification
…rcours Mot de passe oublié [REMANIEMENT] Enleve du inline style pour centrer des textes et utilise une classe CSS [REMANIEMENT] Rajoute la propriété alt sur une illustration [REMANIEMENT][FORMULAIRE MOT DE PASSE OUBLIE] Affiche dans le message de succès l'email renseigné dans le formulaire [REMANIEMENT][FORMULAIRE MOT DE PASSE OUBLIE] Navigue via react-router vers la racine [REMANIEMENT] Renomme des variables prev en precedent [REMANIEMENT] Echappe les caractères spéciaux dans le formulaire réinitialiser mot de passe [REMANIEMENT] Trime le token lors de la vérification si il existe ou non
…ntre un formulaire et son action vers un endpoint
firefly33
force-pushed
the
pages-reinitialisation-mot-de-passe
branch
from
1cdc254
to
e8faace
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.