Development #6710
Remplacer le jsonfield des paramètres de service par un vrai formulaire
0%
Description
Pour le moment il y a un jsonfield pour les paramètres des services de paiement, ce serait bien de ne pas inclure ce champ dans le formulaire de base, mais d'avoir un second formulaire d'options, avec des vrais champs (cf module.description['parameters'], on peut aussi voir ce qui se faisait dans auquotidien).
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 4 ans
- Dupliqué par Development #41622: UI de configuration des plateformes de paiement ajouté
Mis à jour par Valentin Deniaud il y a presque 4 ans
- Lié à Development #43030: adaptations des paramètres des backends, pour y associer des champs ajouté
Mis à jour par Valentin Deniaud il y a presque 4 ans
- Patch proposed changé de Non à Oui
- Fichier 0001-lingo-use-form-instead-of-json-input-for-backends-67.patch 0001-lingo-use-form-instead-of-json-input-for-backends-67.patch ajouté
- Fichier 0002-DONT-merge-eopayment-wip-branch-tox.patch 0002-DONT-merge-eopayment-wip-branch-tox.patch ajouté
- Statut changé de Nouveau à Solution proposée
La structure était déjà toute là grâce à Manu, merci Manu.
Mis à jour par Valentin Deniaud il y a presque 4 ans
Et reste tout de même la question des traductions. Dans eopayment il y a des gettext mais j'ai l'impression qu'ils ne font rien, du coup j'ai rajouté des gettext aussi côté lingo (et là ça marche pour les rares chaînes de eopayment qui ont une trad).
Mis à jour par Frédéric Péters il y a presque 4 ans
Appeler gettext ici va passer les chaines d'eopayment dans les .po de combo, ça ne va pas le faire; j'ai créé #43082.
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
Valentin Deniaud a écrit :
Et reste tout de même la question des traductions. Dans eopayment il y a des gettext mais j'ai l'impression qu'ils ne font rien, du coup j'ai rajouté des gettext aussi côté lingo (et là ça marche pour les rares chaînes de eopayment qui ont une trad).
C'est pas évident à moins de faire dépendre eopayment de Django pour utiliser ses outils de traduction. Il y a un concept de locale courante en Django qui n'est pas le même que dans du code gettext "classique" (parce que je pense que celui dans gettext n'est pas threadlocal, enfin à vérifier, ce qui est embêtant si Django utilise des thread ou des green threads).
Faudrait arriver à dire à Django d'intégrer des traductions d'eopayment dans son catalogue, faire en sorte qu'eopayment ne retourner que des chaînes pas traduite puis à l'usage les passer à gettext ou gettext_lazy; cet avant dernier point n'est pas difficile, car c'est déjà le cas il me semble.
Mis à jour par Valentin Deniaud il y a presque 4 ans
- Fichier 0001-lingo-use-form-instead-of-json-input-for-backends-67.patch 0001-lingo-use-form-instead-of-json-input-for-backends-67.patch ajouté
Vous avez pas l'air d'accord, alors laissons les traductions hors de ce ticket, on fera une réunion. J'ai viré les appels à gettext, trads ou pas ce ticket c'est de toute façon le jour et la nuit par rapport à l'existant.
Mis à jour par Frédéric Péters il y a presque 4 ans
Sur validation du formulaire, si le backend change, je réafficherais la page avec le message qu'on a à la création "Please fill additional backend parameters" (et j'ajouterais un point à ce message).
Mis à jour par Frédéric Péters il y a presque 4 ans
J'ai basculé sur une régie "systempay" et il y a des contrôles sur les champs et je ne peux m'en sortir même en changeant le service de paiement; j'imagine du coup que la validation des champs se fait par rapport au type de service de paiement qui avait été enregistré avant. Je proposerais, pour répondre en même temps à mon commentaire précédent, que le champ "service de paiement" ne puisse simplement pas être modifié après coup.
Mis à jour par Valentin Deniaud il y a presque 4 ans
- Fichier 0001-lingo-use-form-instead-of-json-input-for-backends-67.patch 0001-lingo-use-form-instead-of-json-input-for-backends-67.patch ajouté
Je ne vois pas ce qui pourrait provoquer ce genre de bug, mais je suis bien d'accord que la bonne façon de faire c'est de ne pas permettre de changer le service.
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Go.
Mis à jour par Valentin Deniaud il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 82261bf4046bbe2484340371407e6fdc401f5dca
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Mon May 18 18:31:03 2020 +0200
lingo: use form instead of json input for backends (#6710)Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
lingo: use form instead of json input for backends (#6710)