DamianBermudez94 / desafio-final-mod-9

dwf-m9-desafio-final.vercel.app
0 stars 0 forks source link

Review #1

Open FerDR89 opened 2 years ago

FerDR89 commented 2 years ago

Buenas Damián, ¿Cómo va?. Antes que nada perdón en la demora de la corrección del desafío pero estuve un poco complicado para sentarme a revisarlo.

Probé el primer endpoint con mi mail, también probé con el tuyo, y me responde: Un estado 400 y como mensaje "error": "No se pudo enviar el código"

https://dwf-m9-desafio-final.vercel.app/api/me/address Este endpoint me devuelve tmb un 400 con el siguiente mensaje

 "message": {
        "value": "address",
        "path": "",
        "type": "matches",
        "errors": [
            "la url debe ser /email o /name"
        ],
        "params": {
            "value": "address",
            "originalValue": "address",
            "path": "",
            "regex": {}
        },
        "inner": [],
        "name": "ValidationError",
        "message": "la url debe ser /email o /name"
    }
}

Este tiene el mismo error (En ambos creo que estas teniendo un inconveniente con YUP al momento de validar los datos) https://dwf-m9-desafio-final.vercel.app/api/me

https://dwf-m9-desafio-final.vercel.app/api/search?q=sofa&offset=1&limit=10 Acá el producto que dejaste en la query de la collection no lo encuentra y me responde el results vacío. Probé con el nombre del producto que dejaste en la búsqueda por id y tampoco me esta funcionando.

https://dwf-m9-desafio-final.vercel.app/api/order?productId=rec3NXy2uMR4f4dwE Al querer crear la orden me devuelve un error en donde me solicita indicar el precio unitario, trate de pasarle este body pero no tuve caso:

{ "unit_price":20,
     "quantity":10}

Cualquier duda, nos juntamos en Discord un rato y lo repasamos

DamianBermudez94 commented 2 years ago

Buenas Fer!! Ahí chequee y en revienta todo en la versión producción, versión local joya

FerDR89 commented 2 years ago

Buenas Fer!! Ahí chequee y en revienta todo en la versión producción, versión local joya

Buenas Dami, Encontré un posible error en este endpoint:

PATCHupdate addres https://dwf-m9-desafio-final.vercel.app/api/me/name

Acá deberías poder modificar la dirección y estas esperando que modifique el nombre.

Otro tema que vi es que no se actualiza nunca el estado de la orden, una vez que completo el flujo de pagos debería de actualizarse dicho estado y reflejarlo cuando obtengo nuevamente la orden (mismo podes agregarle la funcionalidad que me avise por mail el estado del pago, etc, etc)

Fijate esas cosillas y cuando las tengas resultas avísame así lo apruebo.

DamianBermudez94 commented 2 years ago

Hola fer!! Ahi mande de nuevo el desafio, ahi chequie todo y a mi me funciono todo =)

FerDR89 commented 2 years ago

Buenas Dami,

Esto es lo que te digo del status de las ordenes:

image

image

image

Tampoco me esta llegando el correo de que lo pagué.

DamianBermudez94 commented 2 years ago

Buenas fer!! Ahi estuve viendo las correcciones y los email al finalizar la compra los envia, cambio de estado, se me esta complicando

DamianBermudez94 commented 2 years ago

El emial lo envia cuando confirmas la compra con mercado pago

FerDR89 commented 2 years ago

El emial lo envia cuando confirmas la compra con mercado pago

Hola Dami, fijate si te sirve de ayuda este ejemplo de como lo hice yo: https://github.com/FerDR89/e-commerce-be/blob/master/pages/api/webhooks/mercadopago.ts

Hice nuevamente un par de compras y no me llega el mail :c Te debería de llegar algo así: image

DamianBermudez94 commented 2 years ago

Hola fer!! lo que me esta pasando es que cuando hago el pago, no me envia el email, solo cuando confirmo la compra con el endpoint de mercado pago

DamianBermudez94 commented 2 years ago

Y cuando confirmo la compra con el endpoint de mercado pago, tambien me cambia la orderId