search

betagouv / analyse-flux-insertion

Outil d'analyse des flux et échanges de données dans le domaine de l'insertion
2 stars 1 forks source link

Ajout de la prise en charge des utilisateurs sans email #61

Closed qblanc closed 3 years ago

qblanc commented 3 years ago

Issue #18 Pour permettre la prise en charge des utilisateurs sans email, cette PR sépare la colonne "Invitation" en "Lien d'invitation" et "Code d'invitation". la première est remplie d'une url si l'utilisateur est doté d'un email valide, et servira de signal au département pour l'envoi d'un mail d'invitation. La seconde est remplie d'un code d'invitation si l'utilisateur n'a pas de mail, a un email non valide, ou si un compte avec son email existe déjà, et servira de signal au département pour l'envoi d'un courrier postal d'invitation.

qblanc commented 3 years ago

Donc ce n'est pas que ce que j'ai fait ne va pas, mais il faut que je le défasse pour le refaire dans une autre PR ?

aminedhobb commented 3 years ago

Donc ce n'est pas que ce que j'ai fait ne va pas, mais il faut que je le défasse pour le refaire dans une autre PR ?

@qblanc non je ne l'ai pas review et il y aurait certainement des choses à changer, mais du coup on verrait ça dans une autre PR.

qblanc commented 3 years ago

mais quel intérêt de ne pas la review maintenant ? En l’occurrence il s'agit d'utilisateurs sans emails, ou sans emails valides, donc cela va parfaitement dans la logique de cette PR. Le fait qu'ils soient des proches ou non n'importe pas, puisqu'ils sont considérés ici en fonction de leur "situation" par rapport à leur mail : on ne peut pas leur créer un compte avec un mail, donc on leur en crée un sans mail.

qblanc commented 3 years ago

Merci pour ta review @aminedhobb ! À part ça, je ne vois pas bien l'intérêt de revenir sur le sujet, mais puisque tu le relances, je n'ai jamais dit que c'était essentiel que tu review l'autre partie ; j'ai sans doute réagi de manière un peu trop émotionnelle, mais simplement, si tu juges qu'une partie de mon travail n'a pas sa place dans une PR, je préfère que tu me challenges sur le pourquoi ou le comment plutôt que de balayer ce travail d'un simple "on a pas besoin de gérer ce cas" ou "on verra ça dans une autre PR" 🙂 Bon week-end et à mardi !

aminedhobb commented 3 years ago

À part ça, je ne vois pas bien l'intérêt de revenir sur le sujet, mais puisque tu le relances, je n'ai jamais dit que c'était essentiel que tu review l'autre partie ; j'ai sans doute réagi de manière un peu trop émotionnelle, mais simplement, si tu juges qu'une partie de mon travail n'a pas sa place dans une PR, je préfère que tu me challenges sur le pourquoi ou le comment plutôt que de balayer ce travail d'un simple "on a pas besoin de gérer ce cas" ou "on verra ça dans une autre PR" 🙂 Bon week-end et à mardi !

@qblanc je disais ça sur le ton de la rigolade mais c'était pas évident à voir, my bad 😬 . Pas de soucis 👍 .