search

TransbankDevelopers / transbank-plugin-woocommerce-webpay

OBSOLETO: Código fuente de Plugin WooCommerce para Transbank Webpay
BSD 3-Clause "New" or "Revised" License
19 stars 17 forks source link

Correción ortográfica y mejora en el flujo final #157

Closed lesterfibla closed 3 years ago

lesterfibla commented 4 years ago

Corregí un error ortográfico con la palabra "Débito". Cambié el orden de ejecución de algunos comandos en el flujo de finalización de venta para que el cambio de estado de la orden woocomerce ocurra después de que se guarden los datos de la transacción que devuelve transbank

gdespirito commented 4 years ago

Gracias por el PR @lesterfibla ! ¿A qué se debe el cambio? Me refiero ¿que diferencia hay entre guardar el status antes o después de guardar los datos?

Saludos!

lesterfibla commented 4 years ago

Hola Gonzalo. Pasa que al momento de hacer el cambio de status de la orden en woocommerce, se gatillan los hooks relativos al cambio de estado, por ejemplo: woocommerce_order_status_changed woocommerce_order_status_processing woocommerce_order_status_completed Entonces, si en esos hooks necesito usar los datos de la órden, incluyendo la información de la venta que devuelve transbank, es necesario que dichos datos existan (que esté guardado el post meta) antes de hacer el cambio de estado. Por ejemplo, en mi caso, tuve que hacer un plugin para un cliente que necesitaba enviar la información de cada nueva compra a su erp, y un dato importante para su asiento contable era el código de autorización de transbank.

gdespirito commented 3 years ago

@lesterfibla disculpa la demora y nunca aprobar esto. Este plugin SOAP está ahora deprecado en reemplazo del nuevo plugin REST: https://github.com/TransbankDevelopers/transbank-plugin-woocommerce-webpay-rest

El código de ese plugin está basado en el de este, así que si la mejora sigue aplicando, feliz de aprobar el cambio si haces el PR en ese repositorio.

Muchas gracias por tu aporte!