Development #26960
paybox: ajout de la possibilité d'annulation et validation d'une transaction
0%
Description
Via les solutions "Paybox Direct" ou "Paybox Direct Plus".
Doc:
Fichiers
Demandes liées
Révisions associées
paybox: implement transaction validation and cancelling (#26960)
debian: add tests required dependencies (#26960)
Historique
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0002-paybox-add-support-for-payment-cancelling-26960.patch 0002-paybox-add-support-for-payment-cancelling-26960.patch ajouté
- Fichier 0001-paybox-return-backend-reference-and-transaction-numb.patch 0001-paybox-return-backend-reference-and-transaction-numb.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Après longue lecture de la doc et appel au support Paybox, j'ai vu coder et tester l'annulation.
Pour la faire il est nécessaire de demander 2 paramètres supplémentaires en retour du paiement: numero de transaction et d'appel générés par Paybox.
Rajouter un attribut "cle" à Payment qui est en fait le mot de passe d'accès au backoffice paybox, nécessaire avec les paramètres de plus haut pour annuler une transaction qui n'a pas encore été débitée.
Mis à jour par Frédéric Péters il y a plus de 5 ans
'caption': _('Clé du site'),
Il y a de la merde dans ce module mais inutile de la répéter. Les appels à gettext, c'est pour les chaines à traduire.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Lié à Bug #25939: paybox : ne pas marquer pour traduction des chaines en français ajouté
Mis à jour par Frédéric Péters il y a plus de 5 ans
response = requests.post(url, params) logger.debug('received %r', response.content) return response
C'est impossible que ça échoue ? (réponse : non, donc gérer ça et lever des ResponseError()).
Mis à jour par Frédéric Péters il y a plus de 5 ans
À part ça, c'est mal défini et gagnerait à se normaliser mais a minima, le retour de cancel() doit être un dictionnaire, pas un objet Response.
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-paybox-add-support-for-payment-cancelling-26960.patch 0001-paybox-add-support-for-payment-cancelling-26960.patch ajouté
Ça peut échouer en effet si la transaction a déjà été annulée.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
À refactorer avec le #27270 pour ne pas avoir ça en deux commits, référencer les deux tickets dans le commit.
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-paybox-implement-transaction-validation-and-cancelli.patch 0001-paybox-implement-transaction-validation-and-cancelli.patch ajouté
- Sujet changé de paybox: ajout de la possibilité d'annulation d'un paiement à paybox: ajout de la possibilité d'annulation et validation d'une transaction
Ok, les deux opération dans un seul commit.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Je pourrai faire chier pour les autres magic number (0000, 00104) mais je le ferai pas :)
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Lié à Development #27270: paybox: possibilité de valider une autorisation de débit ajouté
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
- Assigné à mis à Serghei Mihai
Même si tu ne le fais pas, je les ai rajouté:
commit 18eef0578f055ebc3fb862481fcbe5d7b66e3356 (origin/master, origin/HEAD) Author: Serghei Mihai <smihai@entrouvert.com> Date: Tue Oct 9 16:43:41 2018 +0200 paybox: implement transaction validation and cancelling (#26960) commit 383109078d67ed91a6c84dc3e658dd6cd9bada5b Author: Serghei Mihai <smihai@entrouvert.com> Date: Wed Oct 3 11:00:40 2018 +0200 paybox: return backend reference and transaction numbers in payment response (#26960)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à En cours
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-debian-add-tests-needed-dependencies-26960.patch 0001-debian-add-tests-needed-dependencies-26960.patch ajouté
- Statut changé de En cours à Solution proposée
Zappé les dépendances pour la construction du paquet.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
Heu non, python-mock dans le debian control pas bon, dans tox.ini plutôt.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Parlé trop vite, le build lance les tests.
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit c6920d294e3dc8b34acad42874c83e9de25957c1 (origin/master, origin/HEAD) Author: Serghei Mihai <smihai@entrouvert.com> Date: Fri Oct 12 13:58:28 2018 +0200 debian: add tests required dependencies (#26960)
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
paybox: return backend reference and transaction numbers in payment response (#26960)