search

whooohq / whq-woocommerce-chilexpress-shipping

Chilexpress Shipping method for WooCommerce / Agrega Chilexpress como método de envío para WooCommerce
https://wordpress.org/plugins/woo-chilexpress-shipping/
GNU General Public License v2.0
23 stars 12 forks source link

Cambios para poder seleccionar nueva API Rest de Chilexpress, aunque sea solo en ambiente DEV #235

Closed PatrickCaneloDigital closed 2 years ago

PatrickCaneloDigital commented 2 years ago

Hola Esteban,

Mira, como conversado esto sería mi propuesta para integrar los servicios REST de CHilexpress en el plugin; aunque sea solo en ambiente DEV.

Como conversado en el issue, entiendo bien la problematica de desarrollar algo para una APi contra la cual no podemos verificar nada productivo. Aún así considero que por último dar la opción a los usuarios de poder elegir, y por ultimo que funcionen con la API REST en ambiente desarrollo, puede ser una opción valida.

Sea como sea, en realidad te dejo la propuesta y lo dejo a tu voluntad también si lo quieres integrar en el plugin oficial o no. Entiendo si lo quieres dejar así. Mas que nada quería hacer el ejercicio para ver que tanto código había que tocar, etc.

Esta solución está pensada para funcionar opcionalmente con SOAP o Rest. En el caso rest el selector de ambiente queda el mismo como para la API SOap, es decir la API soap hay que manejarla en WS Prod y la API Rest en WS QA. Por si acaso para efectos de prueba la API REST cobertura no necesita API Keys, la API cotizador si los necesita en dev y prod.

Lo dejo a tu juicio...

Ah, importante (!): para la API Rest no hice la implementación de los transientes... (eso eventualmente habría que implementarlo) y por el otro lado, obvio pero igual lo menciono: Por favor verificar/probar los cambios ;-) los probé localmente y el cambio entre api soap y rest al menos a mi me funcionó bien...

Un abrazo desde Valdivia

TCattd commented 2 years ago

Agradecido del trabajo, @PatrickCaneloDigital . De verdad. El haber incluido opción para usar SOAP o REST, super bien. Super bien hecho.

No soy muy fan de no usar lo que WP ya entrega. Hubiese preferido wp_remote_get a el uso de cURL, por ejemplo. De esa forma, se puede aprovechar todo el "framework" que es WP y, entre otras cosas, cachear esos resultados de esos request en masa.

Voy a probar localmente apenas pueda, y te aviso como me va. Me va a quedar confiar en la respuesta de la cobertura no más y que el resto está bien, porque llaves para probar el cotizador en producción no tengo.

Esteban

PatrickCaneloDigital commented 2 years ago

De nada @TCattd

No soy muy fan de no usar lo que WP ya entrega. Hubiese preferido wp_remote_get a el uso de cURL, por ejemplo.

Si, de acuerdo... es que necesitaba una implamentación rápida y funcional y no me quise tomar el tiempo para revisar al fondo la funcionalidad WP para eso, por lo mismo tampoco implementé los transientes (aún) De hecho el cliente el cual me pidió la actualización me comentó que en producción le funcionaba, así que por ese lado estamos oka (creo jajaja)

Pero se puede incluir en esta o una futura actualización del plugin. Quedo atento a cualquier novedad. y vamos mirando...

Un abrazo