-
Notifications
You must be signed in to change notification settings - Fork 58
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
[BUGFIX] Encoder l'adresse email présente dans l'URL (PIX-17205). #11852
[BUGFIX] Encoder l'adresse email présente dans l'URL (PIX-17205). #11852
Conversation
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
api/src/identity-access-management/domain/emails/create-warning-connection.email.js
Outdated
Show resolved
Hide resolved
2488a53
to
96f7c3c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ Lu et testé fonctionnellement avec succès avec Firefox 🦊
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Juste un petit commentaire ;)
api/tests/identity-access-management/unit/domain/emails/create-warning-connection.email.test.js
Outdated
Show resolved
Hide resolved
96f7c3c
to
9e6ca27
Compare
9e6ca27
to
9f8760f
Compare
🌸 Problème
Je reçois mon e-mail de notification de reconnexion, je clique sur le bouton pour réinitialiser mon mot de passe, mon adresse e-mail est bien reportée mais le “+” n’est pas reporté.
Cette PR fait suite à #11753.
🌳 Proposition
Encoder le contenu du query parameter email.
🤧 Pour tester