Projet

Général

Profil

Development #9280

envoyer l'id et la date de la transaction au webservice de notification de paiement d'une facture

Ajouté par Serghei Mihai il y a plus de 8 ans. Mis à jour il y a presque 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
09 décembre 2015
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Fichiers

Historique

#1

Mis à jour par Serghei Mihai il y a plus de 8 ans

Appel du webservice de notification de paiement en POST d'abord, chose qu'on devait faire des le debut.

#2

Mis à jour par Frédéric Péters il y a plus de 8 ans

Je ne comprends pas très bien à quelle demande pratique ça répond; et quant à la technique, j'imagine que c'est lié à des modifs dans un connecteur passerelle (?).

#3

Mis à jour par Serghei Mihai il y a plus de 8 ans

Actuellement la notification de paiement d'une facture se fait par un simple GET en passant dans l'url l'identifiant de la facture et rien d'autre.

Dans certains cas, notamment pour Alfortville, il est nécessaire d'envoyer l'identifiant et la date de la transaction.

Je propose d'envoyer ces données dans le payload json.

J'adapterai le connecteur Teamnet en conséquence.

#4

Mis à jour par Serghei Mihai il y a plus de 8 ans

  • Fichier 0001-add-transaction-id-and-date-to-invoice-payment-notif.patch ajouté

Les deux patchs fusionnés.
Le numéro de la facture est désormais envoyé dans le payload json et n'est plus spécifié dans l'url du webservice.

#5

Mis à jour par Thomas Noël il y a plus de 8 ans

« requests.post(self.signed_url(request, url), json=data) » le json= ça va pas passer avec le requests qu'on utilise

#6

Mis à jour par Serghei Mihai il y a plus de 8 ans

  • Fichier 0001-add-transaction-id-and-date-to-invoice-payment-notif.patch supprimé
#8

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

Ça peut être sympa de passer un headers={'content-type': 'application/json'} quand on poste du JSON.

#9

Mis à jour par Serghei Mihai il y a plus de 8 ans

Oui.
A garder en tête qu'il faudrait virer ce "hack" lors du passage en jessie

#10

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

Bof ça marchera encore, occupons nous plutôt de ce qui ne marchera plus :)

#11

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

Si c'est testé c'est bon pour moi.

#12

Mis à jour par Serghei Mihai il y a plus de 8 ans

  • Statut changé de En cours à Résolu (à déployer)
commit f497a9c8adaf1c78dd8d6b6ee6dcf1d496e17a61
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Wed Dec 9 16:19:44 2015 +0100

    add transaction id and date to invoice payment notification (#9280)

    Send data through POST
#13

Mis à jour par Frédéric Péters il y a presque 8 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF