Add_fields - Githubissues

msaustral commented 4 years ago

Se agrega en cada pedido las cantidades de cuotas, tipo de tarjeta (Débito o Crédito), código de autorización de Transbank y número de transacción de Transbank para poder llevar su control al cruzarlo con los reportes de Transbank.

fjdiazt commented 4 years ago

hola, este PR seria muy util para el resto de los usuarios. Hace falta algo mas para aprobarlo? Gracias

msaustral commented 4 years ago

hola, este PR seria muy util para el resto de los usuarios. Hace falta algo más para aprobarlo? Gracias

Hola buen día, el código que subimos con los cambios fue eliminado, no entendemos porqué, saludos.

gdespirito commented 4 years ago

Gracias por este PR @msaustral. Realmente es algo que suma. Revisando el código, me preocupa que se edite directamente la tabla de order_payment. Acá la solución ideal es crear una nueva tabla para guardar esa información. Eso evitaría problemas de compatibilidad. ¿Crees que nos puedas apoyar con eso? Si no, podemos revisar como abordar esto nosotros en algun momento.

msaustral commented 4 years ago

Gracias por este PR @msaustral. Realmente es algo que suma. Revisando el código, me preocupa que se edite directamente la tabla de order_payment. Acá la solución ideal es crear una nueva tabla para guardar esa información. Eso evitaría problemas de compatibilidad. ¿Crees que nos puedas apoyar con eso? Si no, podemos revisar como abordar esto nosotros en algun momento.

Hola el tema es que todos los datos están ya en la tabla order_payment, crear otra tabla es redundar en datos.

gdespirito commented 4 years ago

@msaustral, más que redundar todos los datos, la nueva tabla debería ser algo como webpay_orders, donde esta tenga el ID del order_payment y tena solo datos relacionados a webpay (el autorization code, el response code, vci, etc).

msaustral commented 4 years ago

@msaustral, más que redundar todos los datos, la nueva tabla debería ser algo como webpay_orders, donde esta tenga el ID del order_payment y tena solo datos relacionados a webpay (el autorization code, el response code, vci, etc).

Hola, perfecto, hay un par de detalles más que hay que tomar en cuenta:

Para que se puedan ver estos datos adicionales, hay que modificar la plantilla del pedidos del Backoffice y las plantillas PDF del comprobante de transacción (pedido)
Algo que nunca logramos resolver, cuándo regresa la transacción aprobada de TBK a PS, al parecer, TBK es quien envía el correo con el comprobante al cliente y los datos adicionales no se muestran, como si no tomara los datos guardados en la BD, pero si en el pedido, en el backoffice, le das reenviar correo si llega el comprobante con todos los datos.

Saludos.

fjdiazt commented 4 years ago

@msaustral mi sugerencia es que se introduzca esta mejora en varias fases. Como fase inicial se podrian almacenar los datos en una tabla separada asociada al pedido correspondiente. Esta info en principio seria para auditar, su acceso seria por consultas en el SQL manager.

Como segunda fase se le podria dar una interfaz a nivel de la plantilla del backoffice o incluso para hacerlo mas sencillo, a nivel de modulo (en el module setup)

Con respecto a tu ultimo comentario sobre el envio de correos, sin conocer el funcionamiento del modulo, me parece que hay un tema de cache a nivel de ese proceso, estos datos se deben almacenar en cache al comienzo del proceso y el correo es enviado en base a estos datos, como la info adicional de pago se guarda luego, no se encuentra en el cache, y por ende no es enviada. Cuando reenvias el correo, toma los datos porque es una nueva consulta y ahi si tiene la info.

msaustral commented 4 years ago

@msaustral mi sugerencia es que se introduzca esta mejora en varias fases. Como fase inicial se podrian almacenar los datos en una tabla separada asociada al pedido correspondiente. Esta info en principio seria para auditar, su acceso seria por consultas en el SQL manager.

Como segunda fase se le podria dar una interfaz a nivel de la plantilla del backoffice o incluso para hacerlo mas sencillo, a nivel de modulo (en el module setup)

Con respecto a tu ultimo comentario sobre el envio de correos, sin conocer el funcionamiento del modulo, me parece que hay un tema de cache a nivel de ese proceso, estos datos se deben almacenar en cache al comienzo del proceso y el correo es enviado en base a estos datos, como la info adicional de pago se guarda luego, no se encuentra en el cache, y por ende no es enviada. Cuando reenvias el correo, toma los datos porque es una nueva consulta y ahi si tiene la info.

hola perfecto, totalmente deacuerdo, saludos

gdespirito commented 4 years ago

@msaustral muchas gracias por este PR. En la nueva versión del plugin se creó una nueva tabla llamada webpay_orders que guarda toda esta información. Lo que queda pendiente es hacer una interfaz en la administración para poder revisar estos datos de manera más amigable para el usuario. Debido a que este método de guardar la información es incompatible con la nueva modalidad, doy por cerrado este PR, pero no sin volver a agradecer el apoyo con esto!

🎉

msaustral commented 4 years ago

@msaustral muchas gracias por este PR. En la nueva versión del plugin se creó una nueva tabla llamada webpay_orders que guarda toda esta información. Lo que queda pendiente es hacer una interfaz en la administración para poder revisar estos datos de manera más amigable para el usuario. Debido a que este método de guardar la información es incompatible con la nueva modalidad, doy por cerrado este PR, pero no sin volver a agradecer el apoyo con esto!

🎉

Hola Gracias a ustedes, es un gusto poder aportar, saludos.

TransbankDevelopers / transbank-plugin-prestashop-webpay

Add_fields #56