betagouv / betaGouvBot

Automated assistant for beta.gouv.fr administrative tasks on slack.
https://beta.gouv.fr
MIT License
4 stars 10 forks source link

Request badges for new members #7

Closed Morendil closed 7 years ago

Morendil commented 7 years ago

Current behaviour:

This may be good enough to go live. Future enhancements will include a better representation of the employer field, and perhaps explicit handling of contract renewals.

MattiSG commented 7 years ago

Ça n’avait pas été ajouté pour le fun, tu dois préciser si c’est presta ou non, c’est important pour leur registre. On peut tenter sans, mais c’était une demande explicite côté administration.

Morendil commented 7 years ago

Argh. Je vais revert le commit précédent.

bonjourmauko commented 7 years ago

Je me demande s'il est réellement nécessaire d'ajouter la dépendance à redis et à quelle point elle est lourde, mais je ne m'y connais pas suffisamment sur le sujet dans l'immédiat et n'ai pas le temps de rechercher pour avoir un avis.

Le filesystem de Heroku est éphémère et ne permet pas d'écrire au disque, donc ça rend obligatoire d'écrire à une base de données qui ne soit pas server-less. Lorsque le job-to-be-done est de « ne pas être harcelé·e (en tant que personne responsable de faire de badges) par des demandes (répétées) de ceux-ci », il est besoin IMHO de savoir si l'email a été déjà envoyé avant.

Ceci dit, j'imagine rapidement les solutions :

Morendil commented 7 years ago

Before merging: