Projet

Général

Profil

Development #36322

tipi: refdet devrait aussi aller dans order_id

Ajouté par Benjamin Dauvergne il y a plus de 4 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
20 septembre 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

transaction_id/order_id sont confondus pour tipi (et lingo nécessite un order_id).


Fichiers

Révisions associées

Révision d7889675 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 4 ans

tipi: confound order_id/transaction_id with refdet (#36322)

TIPI has no concept of a transaction_id and our main user (lingo
application in combo) use only order_id. PaymentResponse documentation
should be changed to deprecate transaction_id, and first return value of
.request() should be renamed order_id to make things understandable as
with all backend what you get in PaymentResponse.order_id is what is
returned by backend.request()[0].

Historique

#2

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

  • Assigné à mis à Benjamin Dauvergne
#3

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

Voilà, si je pouvais je referai une API plus claire mais là bon...

#4

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

  • Statut changé de Solution proposée à Solution validée
#5

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit d7889675b7be5d5622ee90ee697cf4245dd72cc4
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Sat Sep 21 17:10:01 2019 +0200

    tipi: confound order_id/transaction_id with refdet (#36322)

    TIPI has no concept of a transaction_id and our main user (lingo
    application in combo) use only order_id. PaymentResponse documentation
    should be changed to deprecate transaction_id, and first return value of
    .request() should be renamed order_id to make things understandable as
    with all backend what you get in PaymentResponse.order_id is what is
    returned by backend.request()[0].
#6

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

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF