Produits Entr'ouvert » Authentic 2

• utiliser le template de base
• utiliser le template de base pour l'OU de l'utilisateur (via le slug)
• utiliser un template spécifique pour l'import CSV
• utiliser un template spécifique pour l'import CSV et l'OU de l'utilisateur (via le slug)

Pour les mots de passe : send_reset_password_reset_email(), qui sera utilisé, fera un set_password(random) donc pas de souci de ce coté là, par contre ce serait utile d'avoir un mode set_random_password simple si on veut par ailleurs géré le mailing autrement.

Donc une colonne '@registration' avec deux valeurs possibles : send-email ou set-random-password.

Benjamin Dauvergne a écrit :

ce serait utile d'avoir un mode set_random_password simple si on veut par ailleurs géré le mailing autrement.

Je comprends pas cette deuxième option, ça sert à quoi de mettre un mot de passe aléatoire par rapport à ne pas en mettre ? Quel serait le cas d'usage concret ?

Si un utilisateur n'a pas de mot de passe, le lien de récupération de mot de passe ne fonctionne pas donc a minima faut en mettre un même random et inconnu de tout le monde si tu veux pouvoir faire un mailing aux gens en disant : allez sur la page de login et faite une récupération de mot de passe.

Compris, je ne savais pas.

Deuxième interrogation, je comprends pas tout à la mécanique de templates que tu imagines. J'ai pigé pour OU/pas OU (merci Paul), mais dans quel cas tu imagines un recours au template par défaut ? Je trouverais pas ça clair de recevoir un mail disant en substance « bonjour vous avez demandé une réinitialisation de votre mot de passe » si on vient de créer mon compte, donc pour moi template spécifique => always.

En fait il y a déjà templates/authentic2/api_user_create_registration_email qui se rapproche plus de ce qu'on voudrait envoyer, à voir si le « À votre demande » est problématique ou pas.

Revoilà un patch 1 avec le boulot dans send_templated_mail.

Valentin Deniaud a écrit :

Avec le patch c'est mieux.

Je note tout de suite que ce serait bien d'avoir un test unitaire sur send_templated_mail aussi.

Désolé de t'avoir laissé patauger, je pense qu'encore plus simple tu pourrais poser une configuration des templates en dur dans tests/settings.py pointant vers tests/templates/ et y mettre les différents templates dont tu as besoin.

tests rouges à cause de #37390

Ils l'étaient aussi pour une autre raison, pour ajouter le répertoire tests/templates/ j'avais tenté un from authentic2.settings import TEMPLATES, puis TEMPLATES[0]['DIRS'].append(...), ça semblait marcher mais les tests rbac plantaient pour une raison étrange. Plus qu'à copier coller la config des templates depuis les settings vers les tests settings, en ajoutant le nouveau répertoire. Anyway, c'est vert.

Valentin Deniaud a écrit :

tests rouges à cause de #37390

Ils l'étaient aussi pour une autre raison, pour ajouter le répertoire tests/templates/ j'avais tenté un from authentic2.settings import TEMPLATES, puis TEMPLATES[0]['DIRS'].append(...), ça semblait marcher mais les tests rbac plantaient pour une raison étrange. Plus qu'à copier coller la config des templates depuis les settings vers les tests settings, en ajoutant le nouveau répertoire. Anyway, c'est vert.

Ni besoin de recopier ou d'importer, TEMPLATES est déjà dans le scope (tests/settings.py est déclaré comme variable d'environnement AUTHENTIC2_SETTINGS_FILE qui sert à faire un execfile() dans le contexte de authentic2.settings tout à la fin).

Tout à fait d'accord. Donc c'est reparti pour un renommage de @registration en @send-email, qui devient alors une colonne booléenne ?

Valentin Deniaud a écrit :

J'ai relu le texte du ticket et je me suis aperçu que le deuxième point de Marie avait été mis de côté, alors en même temps que je mettais un mot de passe à tous les utilisateurs créés par défaut, j'ai remplacé l'option qui permettait de le faire par une autre qui correspond à ce point 2.

C'est vraiment pas une bonne idée de proposer ça, je suis pour ignorer cette demande, il y a clairement un "ou" dans ses propositions pour moi il est exclusif.

Non, il n'était pas exclusif, mais je vais clarifier ma demande : ce serait bien d'avoir la possibilité de faire soit l'un, soit l'autre, et que ce choix soit possible par import.

Marie Kuntz a écrit :

Non, il n'était pas exclusif, mais je vais clarifier ma demande : ce serait bien d'avoir la possibilité de faire soit l'un, soit l'autre, et que ce choix soit possible par import.

Et donc je dis que le deuxième est interdit, on n'envoie pas des mots de passe dans des mails qui pourront peut-être y rester et ne jamais changer.

ok, ça me va

Rebasé, j'attends que ce soit vert et je pousse.