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 nouvel instructeur, je dois confirmer mon mail #10473

Open
2 tasks
Tracked by #10424
LeSim opened this issue May 30, 2024 · 0 comments · May be fixed by #10549
Open
2 tasks
Tracked by #10424

ETQ nouvel instructeur, je dois confirmer mon mail #10473

LeSim opened this issue May 30, 2024 · 0 comments · May be fixed by #10549
Assignees

Comments

@LeSim
Copy link
Member

LeSim commented May 30, 2024

On veut

Lors de l'ajout d un instructeur par un admin / par un instructeur

  • on ne confirmed/verified plus son compte par défaut
  • on fusionne le mail de notification d'ajout d un instructeur et d'invitation

On estime dans un premier temps que les adresses d'agent connect sont propres.

Logique existante Instructeur

Ajout par l'admin

lorsqu'un instructeur est ajouté par l'admin de la demarche (admin/groupe_instructeur_controller#add_instructeur) :

  • on crée l'instructeur et on confirmed/verified son compte par défaut
  • une invite est envoyée groupe_instructeur.add_instructeur
  • puis une notification d'ajout est envoyée notify_added_instructeurs

Ajout par un autre instructeur

methode instructeurs/groupe_instructeurs_controller#add_instructeur :

  • on crée l'instructeur et on confirmed/verified son compte par défaut
  • une invite est envoyée dans def create_instructeur
  • puis une notification d'ajout est envoyée notify_added_instructeurs

Création par AgentConnect

  • création de l instructeur dans agent_connect/agent_controller:28, confirmed/verified par défaut
  • pas d'envoi de 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

Successfully merging a pull request may close this issue.

2 participants