get_transaction está fallando

[ragnarok22]

Describe the bug
A la hora de obtener una transacción da error

To Reproduce

client = QvaPayClient(app_id=settings.QVAPAY_APP_ID, app_secret=settings.QVAPAY_APP_SECRET)
transaction = client.get_transaction(transaction_id)

Esto lanza un error en transaction_detail.py en la línea 28 a la hora de crear un PaidBy ya que este espera un username, logo y name pero está recibiendo estos parámetros:

'paid_by': {
    'uuid': 'fasdfasdf',
    'username': 'ragnarok',
    'name': 'Reinier',
    'lastname': 'Hernández',
    'bio': 'I ♥ python',
    'logo': 'profiles/logo.webp',
    'kyc': 0
  },

Adjunto traceback

[leynier]

Muchas gracias por reportar el error, agregaron esas propiedades que antes no estaban. @LugoDev @codeshard @ragnarok22 les parece mejor que la biblioteca no de error si se agregan propiedades nuevas, que solo de error si cambian el formato o eliminan una de las existentes?

[leynier]

@ragnarok22 el uuid que esta poniendo es un ejemplo o es lo que esta devolviendo QvaPay?

[leynier]

Por lo que pone en el issue al parecer el paid_by tiene la misma estructura que owner solo que en owner el uuid es de la forma 796a9e01-3d67-4a42-9dc2-02a5d069fa23, por eso mi pregunta anterior.

[CarlosLugones]

Este error me parece muy extraño, es como si estuvieran realizando cambios a la API. ¿Qué hacemos?

[leynier]

@LugoDev una posible solución es lo que comento, de implementar que la lib solo de error si se cambia o se elimina alguna de las propiedades que se esperan, no como en este caso que da error porque devuelve contenido "extra".

[ragnarok22]

@ragnarok22 el uuid que esta poniendo es un ejemplo o es lo que esta devolviendo QvaPay?

Si, es un ejemplo

[leynier]

Eso provocaría que fuera mas difícil darse cuenta de esos contenidos extras y actualizar la lib, pero dejaría de dar error a los que la utilicen.

[leynier]

Entonces, si están de acuerdo, realizaré un PR solo implementando que de error cuando se hace un cambio o eliminación, no cuando se hace una adición de propiedades a las respuestas que de la API.

[leynier]

Para publicar una especie de hotfix

[CarlosLugones]

Pues me parece una buena solución. No entiendo por qué están cambiando una API que está en producción, no debería pasar, pero es lo que hay.

[CarlosLugones]

De acuerdo Leyinier, gracias por tu ayuda!

[ragnarok22]

perfecto @leynier

[leynier]

Listo chicos @ragnarok22 @LugoDev como cosa nueva agregue lo siguiente.

Si tengo la la instancia a de la clase A con x y y de propiedades, si en el json viene una nueva propiedades z será agregada dinámicamente a la instancia, no tendrá chequeo de tipos, ni auto-completamiento, porque todo sucede en tiempo de ejecución aprovechando el dinamismo de Python, pero si se hace a.z se podrá acceder a la nueva propiedad z.

[leynier]

@all-contributors please add @ragnarok22 for bug

[allcontributors]

@leynier

I've put up a pull request to add @ragnarok22! 🎉

[ragnarok22]

Pregunta: esta versión se lanzó ya para pypi?

[CarlosLugones]

Aún no, tengo pendiente bajarme el repo y probarlo localmente. Si da tiempo, lo lanzo hoy mismo y aviso en Twitter. Revisa la documentación, porque tiene breaking changes. El dom, 31 oct 2021 a las 13:46, Reinier Hernández (< ***@***.***>) escribió:

…

Pregunta: esta versión se lanzó ya para pypi? — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#13 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AEO5SOSNNX4KWFVPFJMPOCLUJV6IRANCNFSM5HAS47KA> .

-- Saludos, Carlos Lugones https://lugodev.com