Development #27814
Maarch: implémenter un WS pour envoyer une réponse
0%
Description
Sur base du WS Maarch "updateStatus".
Fichiers
Révisions associées
Historique
Mis à jour par Pierre Cros il y a plus de 5 ans
- Statut changé de Nouveau à Rejeté
C'est depuis w.c.s. que l'envoi doit être fait
Mis à jour par Frédéric Péters il y a plus de 5 ans
Je lis entre les lignes pour ne pas jouer aux statuts, il y aurait appel d'un webservice par w.c.s., ce webservice serait implémenté dans Welco, qui dispose déjà de code et cie pour se connecter à Maarch.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Fichier 0001-tests-remove-debugging-print.patch 0001-tests-remove-debugging-print.patch ajouté
- Fichier 0002-maarch-implement-sending-a-response-27814.patch 0002-maarch-implement-sending-a-response-27814.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Avec un test, qui passe.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Ça s'appelle ainsi:
- URL : {{welco_url}}api/mail/response/
- Paramètres JSON :
- mail_id : form_submission_context_external_id (pas sûr du nom de la variable)
- content : le contenu qu'on souhaite voir dans historyMessage sur Maarch
Mis à jour par Frédéric Péters il y a plus de 5 ans
'djangorestframework>=3.3, <3.4',
La limitation <3.4 est justifiée par quoi ? (côté chrono on a < 3.7, pour encore tourner en 1.8)
Je serais pour garder les erreurs 500 à de l'imprévu.
Ça a été testé avec Publik ? (je pense surtout à la signature automatique des appels qui peut avoir lieu)
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Je vais tester avec 3.7, j'ai pris la ligne du setup.py de combo qui elle même devait venir initialement d'authentic.
Non pas testé avec Publik, je compte sur la configuration de REST_FRAMEWORK['DEFAULT_AUTHENTICATION_CLASSES'] venant de debian_config_common.py pour faire le job, à ma connaissance on ne teste pas nos APIs avec Publik, si je prends chrono en exemple.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Fichier 0001-tests-remove-debugging-print.patch 0001-tests-remove-debugging-print.patch ajouté
- Fichier 0002-maarch-implement-sending-a-response-27814.patch 0002-maarch-implement-sending-a-response-27814.patch ajouté
- rest_framework <3.7
- erreurs 500 retirés
- plus de tests sur les cas d'erreur
Mis à jour par Frédéric Péters il y a plus de 5 ans
à ma connaissance on ne teste pas nos APIs avec Publik, si je prends chrono en exemple.
Je ne parle pas de test automatisé, juste en local, tester de bout en bout.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Frédéric Péters a écrit :
à ma connaissance on ne teste pas nos APIs avec Publik, si je prends chrono en exemple.
Je ne parle pas de test automatisé, juste en local, tester de bout en bout.
Ah, j'ai pas Publik en local.
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-maarch-implement-sending-a-response-27814.patch 0001-maarch-implement-sending-a-response-27814.patch ajouté
Benjamin Dauvergne a écrit :
Non pas testé avec Publik, je compte sur la configuration de REST_FRAMEWORK['DEFAULT_AUTHENTICATION_CLASSES'] venant de debian_config_common.py pour faire le job, à ma connaissance on ne teste pas nos APIs avec Publik, si je prends chrono en exemple.
Pour qu'elle s'applique il faut rajouter 'rest_framework' dans installed_apps.
Testé en local et ça fonctionne.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Fichier 0001-tests-remove-debugging-print.patch 0001-tests-remove-debugging-print.patch ajouté
- Fichier 0002-maarch-add-a-webservice-for-sending-a-mail-response-.patch 0002-maarch-add-a-webservice-for-sending-a-mail-response-.patch ajouté
Voilà intégré la modif d'INSTALLED_APPS.
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
Voilà poussé taggé.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
maarch: add a webservice for sending a mail response (#27814)