Development #42921
connecteurs SMS, possibilité d'alerte au "niveau d’alerte du crédit d’appel"
0%
Description
Plutôt que retourner l'info dans la réponse de l'endpoint d'envoi d'un SMS et peut-être attendre que tous les workflows prennent en compte ça pour avertir l'admin, avoir un paramètre "email où envoyer l'alerte" et que Passerelle se charge de cet email.
On doit faire attention à ne pas faire un envoi d'alerte à chaque SMS mais pour moi pas besoin de se limiter à l'envoi d'une unique alerte, s'il y a un email par jour, c'est très bien.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Lié à Development #46316: Connecteur SMS OVH : avoir le crédit restant affiché dans le connecteur ajouté
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Assigné à
Valentin Deniaudsupprimé
Le ticket dit « connecteurs sms » mais je ne compte regarder que ovh, c'est aussi le seul où on parle de crédit actuellement.
Mis à jour par Frédéric Péters il y a plus de 3 ans
L'idée est de prévoir la chose au niveau générique, qu'un connecteur SMS ait juste à implémenter une méthode get_credits_info() (genre, j'écris ça sans avoir regardé ce qu'il y a dans le connecteur OVH actuellement).
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier 0001-ovh-send-email-credit-alerts-42921.patch 0001-ovh-send-email-credit-alerts-42921.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Je ne pense pas que tu puisses faire ça :
{% with _("View connector page") as button_label %}
à mon avis tu voulais écrire :
{% trans "View connector page" as button_label %}
Je ne sais pas si c'est vraiment utile de faire un check lors de l'envoi d'un SMS, le cron hourly m'a l'air suffisant (et ça risque d'envoyer plusieurs mails si on envoie rapidement des SMS, et ça met un cas d'erreur possible dans le code d'envoi, genre on a une exception SMTP, autant ne pas gérer ça ici), après c'est simplement qu'il faut utiliser un seuil réaliste pour qu'avoir 1h entre le dépassement du seuile et l'envoi du mail ne soit pas un drame, genre la moyenne des envois sur 1 mois (un ticket intéressant serait d'ailleurs de pouvoir donner ce genre de chiffre de consommation, voir de l'utiliser automatiquement, moyenne mensuelle glissante sur 12 mois).
C'est tout ok pour le reste.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
Benjamin Dauvergne a écrit :
Je ne pense pas que tu puisses faire ça :
Et si (copier coller depuis chrono, où cette syntaxe marche déjà, ©Fred).
Je ne sais pas si c'est vraiment utile de faire un check lors de l'envoi d'un SMS, le cron hourly m'a l'air suffisant (et ça risque d'envoyer plusieurs mails si on envoie rapidement des SMS, et ça met un cas d'erreur possible dans le code d'envoi, genre on a une exception SMTP, autant ne pas gérer ça ici)
Ouais, je vire.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit fc615aff90fc3fc54a6ed02f739982f8124d81b5 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Oct 7 11:31:03 2020 +0200 ovh: send email credit alerts (#42921)
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Valentin Deniaud a écrit :
Benjamin Dauvergne a écrit :
Je ne pense pas que tu puisses faire ça :
Et si (copier coller depuis chrono, où cette syntaxe marche déjà, ©Fred).
Pour la lisibilité c'est dommage que ce ne soit pas documenté et c'est un peu gratuit vu tous les 'trans/blocktrans' autour, si encore tu évitais d'introduire la variable button_label avec un with je comprendrais.
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
ovh: send email credit alerts (#42921)