Development #32967
Commencer les options de transactions sur les régies
0%
Description
Suite de #32441, sur une régie dont le backend de paiement le supporte, offrir la possibilité configurer la régie pour que les transaction qu'elles génèrent aient le mode de validation manuel.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Lié à Development #32441: Nouvel objet PaymentBackend ajouté
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Lié à Development #32969: Surcharger dynamiquement des parametres d'une transaction ajouté
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Fichier 0001-wip.patch 0001-wip.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Histoire de donner un avant goût.
Ça stocke les options d'une régie sur un champ json transaction_option que l'utilisateur ne voit pas, à la place génération dynamique de champs de formulaire
en se basant sur ce les backend proposent (#32969)
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Fichier 0001-lingo-start-transaction-options-32967.patch 0001-lingo-start-transaction-options-32967.patch ajouté
- Sujet changé de Regie de paiement à validation manuelle à Commencer les options de transactions sur les régies
- Statut changé de En cours à Solution proposée
Avec des tests.
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Fichier 0001-lingo-start-transaction-options-32967.patch 0001-lingo-start-transaction-options-32967.patch ajouté
Mis à jour par rapport aux changements dans #32969.
C'est modeste, pas d'ajax et donc un temps de retard en cas de changement de backend, les options permises par le nouveau backend n'apparaissent pas tout de suite il sauver une première fois puis éditer à nouveau ... (je vais voir gérer l'ajax en espérant que ça me prenne pas trois plombes).
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Fichier 0001-lingo-start-transaction-options-32967.patch 0001-lingo-start-transaction-options-32967.patch ajouté
Option de ne pas présenter ces champs dynamiques lors de la création d'une régie (ainsi que les autres champs standards peu utilisés),
tous les champs présents lors de l'édition.
En se disant que changer le backend de paiement d'une régie existante est un cas marginal, et qu'on s'autorise donc à éviter ajax and co.
Également terrain mieux préparé si on veut se mettre à faire la même chose sur les payment backend.
Mis à jour par Frédéric Péters il y a presque 5 ans
(ainsi que les autres champs standards peu utilisés)
J'en serais à également retirer le is_default, je regarde le save() de la Regie et ça me semble pouvoir marcher tout seul.
combo/apps/lingo/forms.py
Y manque le préambule de licence.
if param['type'] is bool:
On mettrait un else: raise NotImplementedError pour être sûr ?
super(RegieForm, self).save()
Je préfère la forme instance = super(RegieForm, self).save(), puis agir sur instance (plutôt que self.instance) plutôt que dépendre du fait qu'en interne l'attribut s'appelle "instance".
Mis à jour par Frédéric Péters il y a presque 5 ans
Je préfère la forme instance = super(RegieForm, self).save(), puis agir sur instance (plutôt que self.instance) plutôt que dépendre du fait qu'en interne l'attribut s'appelle "instance".
Sauf que je me dis que ça ne tient pas la route, que je partais de l'idée que ça retournait l'objet alors que ce n'est peut-être bien pas le cas.
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Fichier 0001-lingo-start-transaction-options-32967.patch 0001-lingo-start-transaction-options-32967.patch ajouté
Tout pris en compte.
Frédéric Péters a écrit :
J'en serais à également retirer le is_default, je regarde le save() de la Regie et ça me semble pouvoir marcher tout seul.
Oui ça marche.
Je préfère la forme instance = super(RegieForm, self).save(), puis agir sur instance (plutôt que self.instance) plutôt que dépendre du fait qu'en interne l'attribut s'appelle "instance".
Ça passe aussi.
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 23f13059d00a761af113a09b92d0031f9648cddb
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Tue May 14 16:38:32 2019 +0200
lingo: start transaction options (#32967)
Based on the backend capabilities.
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
lingo: start transaction options (#32967)
Based on the backend capabilities.