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

ETQ Usager se connectant par FC, je dois confirmer mon mail #10425

Open
Tracked by #10424
LeSim opened this issue May 16, 2024 · 2 comments
Open
Tracked by #10424

ETQ Usager se connectant par FC, je dois confirmer mon mail #10425

LeSim opened this issue May 16, 2024 · 2 comments
Assignees

Comments

@LeSim
Copy link
Member

LeSim commented May 16, 2024

UI

le compte FC est déjà lié à un compte DS

Rien à faire

l'email FC est inconnu de DS

Actuellement : connexion automatique et création de compte

On veut :

Bonjour,

Votre compte FranceConnect utilise `ahah@spamhaus.com` comme email de contact.

Souhaitez-vous l'utiliser pour recevoir les notifications concernant l'avancement de vos dossiers ?

[oui] [non]

  1. si non
Donnez nous alors le mail que ds utilisera pour vous contacter

Email: [ ] # <- utiliser la détection de typo

[Utiliser ce mail]
  1. dans tous les cas on affiche
Un mail de confirmation a été envoyé à votre adresse "ma_bonne@adresse.fr"
Veuillez cliquer sur le lien du mail si vous souhaitez recevoir les notifications concernant l'avancement de vos dossiers.

[Continuer]

l'email FC est connu de DS, mais l'utilisateur souhaite en utiliser un autre

Actuellement :
Screenshot 2024-05-16 at 11-13-19 Fusion des comptes FC et demarches-simplifiees fr · demarches-simplifiees fr

On veut :

  1. l'usager entre son mail dans une interface qui utilise la détection de typo et clique sur utiliser ce mail
  2. on affiche
Un mail de confirmation a été envoyé à votre adresse "ma_bonne@adresse.fr"
Veuillez cliquer sur le lien du mail si vous souhaitez recevoir les notifications concernant l'avancement de vos dossiers.

[Continuer]

Point bonus : on met au style DS FR

Code

On ne remplit plus par défaut email_verified_at du compte usager dans la méthode associate_user! dans FranceConnectInformation.rb en utilisant le callback user.after_confirmation qui remplit email_verified_at.

A la place, on envoie un mail de confirmation user.send_confirmation_instructions (?) et c'est uniquement lors de la confirmation que l'on remplit email_verified_at.

@LeSim LeSim changed the title mail utilisé lors d'une création de compte par FC ETQ Usager se connectant par FC, je dois confirmer mon mail May 16, 2024
@kara22 kara22 self-assigned this Jun 24, 2024
@kara22
Copy link
Contributor

kara22 commented Jun 26, 2024

Salut @LeSim

Aujourd'hui il est possible de modifier son mail en le vérifiant dans Mon profil > Coordonnées > modifier mon mail

Ensuite l'usager doit le vérifier dans son mail, même si il est connecté avec FC, le changement peut se faire.

Si je comprends bien, l'objectif de la feature est : pour ceux qui se connectent avec FC et ou il n'existe pas de compte DS lié, les rediriger vers une page qui leur demande de choisir si le mail est OK ou si ils veulent changer ?

EDIT : j'ai bien compris le sujet en relisant l'issue ;)

@LeSim
Copy link
Member Author

LeSim commented Jun 26, 2024

@kara22

On ne s'occupe pas du changement d'email dans le profil dans cette issue.

Si je comprends bien ...

le code intéressant est dans app/controller/france_connect/particulier_controller.rb#callback

t'as 3 cas :

  • l'identite france connect et un compte ds sont déjà liés (france_connect_information.user.present?) => on ne fais rien
  • ils ne sont pas liés et l'email fci.email_france_connect n'est pas utilisé dans la base
    • c'est le cas l'email FC est inconnu de DS décrit plus haut : on demande a l'utilisateur quel mail il veut choisir et on le prévient que tant qu'il n'aura pas confirmé son email, il ne recevra pas de notifications
  • ils ne sont pas lié et l'email fci.email_france_connect est utilisé dans la base
    • c'est le cas l'email FC est connu de DS, mais l'utilisateur souhaite en utiliser un autre , on utilise le correcteur de typo et il ne recevra pas de notification tant qu'il n'aura pas confirmé son mail.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: En cours
Development

No branches or pull requests

2 participants