Produits Entr'ouvert » Passerelle

Patch qui reprend de l'ancien code avant le passage aux envois asynchrone : https://dev.entrouvert.org/issues/21465

(compris)

• uniformiser l'utilisation du template messages_services_views pour tous les connecteurs SMS
• le déplacer dans /sms/template
• ajouter le lien vers une popup dedans
• ajouter des fichiers urls.py, views.py et forms.py dans /sms
• bidouiller base/__init__.py pour charger sms/urls.py sur les connecteurs SMS
• bidouiller une fonction get_instance() dans la vue pour retrouver l'objet connecteur

• permettre d'appeler le endpoint de façon synchrone
• ajouter mon formulaire directement dans le template messages_services_views
• faire en sorte que le formulaire envoie ses paramètres en JSON sur le endpoint

Bref, je préfère passer mon chemin pour le moment quitte à revenir plus tard.

Un bouton (oups, un lien en fait) "Tester l’envoi" qui ouvre une popup.

get_instance ne peut pas fonctionner comme ça (capitalize()), il faut utiliser GenericConnectorMixin qui te donne self.model.

merci. J'ai dû surcharger init_stuff (avec le patch du mixin proposé les tests plantent sur le connecteur CSV).

L'appel à send_sms n'est pas le même que celui qui est fait par l'envoi normal via BaseSMSResource, il manque le paramètre stop. Ça fait crasher l'envoi avec le connecteur OVH par ex. Bref, juste ajouter un stop=False.

merci, je n'avais pas détecté le soucis en m'envoyant les SMS avec twilio.

       if not kwargs['stop']:   # yep, crash ici
            params.update({'noStop': 1})  # yep, je met stop=False

À l'usage, j'aurais aimé avoir un bouton « Resend test message », parce que remplir le formulaire de test, valider l'envoi, voir que ça ne marche pas, corriger, remplir le formulaire de test avec les mêmes infos, valider l'envoi, etc, c'est un peu relou.

Moi je veux bien, mais c'est pas un peu over-engenering (le navigateur se souvient des valeurs, ça fait 7 clics) ?

Oui, je ne voulais pas toucher au code du Mixin mais c'est plus propre.

Oui et non.

Tu dois pouvoir enlever l'affectation explicite de self.form_class dans le mixin

oui à condition de remplacer l'endroit où cette affectation est utilisée (dans GenericConnectorView) -> 0001

et en fait pas besoin de surcharger get_form_class dans le formulaire SMS, l'attribut statique devrait suffire

non, il faut de toute façon redéfinir get_form_class car elle est appelée par le FormMixin de django.

commit 257334fdc5354773797297eb69ece1ffaac332bb
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Wed Sep 16 16:23:49 2020 +0200

    sms: add a button to test sending sms (#45333)

commit 6ff2dc4365c8511e083ee6c16ee6b91ea691224a
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Thu Oct 15 14:52:35 2020 +0200

    views: explicitly get form_class from generic connector mixin (#45333)