Fix: Webhook events - Githubissues

conekta / conekta_prestashop

12 stars 11 forks source link

Closed edenriquez closed 6 years ago

edenriquez commented 6 years ago

Why is this change neccesary? This change is a bug fix that fixes this issue https://github.com/conekta/conektaprestashop/issues/51#issuecomment-319850128 How does it address the issue?

change webhook creating from ccharge.paid (version 1.0.0) to order.paid (version 2.0.0)
Putting metadata in order details because is used at webhook code in order to change prestashop order status

What side effects does this change have? nothing, better user experience

JoValo commented 6 years ago

Este cambio ya esta probado?

edenriquez commented 6 years ago

si, localmente @JoValo mande un curl con los datos del evento con una orden/evento con estos nuevos atributos

JoValo commented 6 years ago

Estaría bueno comprobar el correcto funcionamiento de los correos