Open EzequielVilla opened 2 years ago
Hola Eze, ahi modifique las cosas que me indicaste !!
En cuanto a lo de las orders en airtable lo incorpore porque el Marce en un video dijo que estaria bueno hacer una table con las ordenes en airtable tambien .
Matiii, ahi estoy viendo los endpoints y voy anotando lo que veo.
No vi ningun otro detalle! Ahora le digo a otro de los chicos que le pegue una ojeada al codigo a ver si nota algo que me perdi!
Mal, no me habia dado cuenta, ahi lo modifique y ahora solo retorna el email con el codigo !!
Bueno Mati! Para mi el codigo asi esta bien y funciona!
Quedo una buena estructura para hacerle unos cambios chiquitos que se vienen. Exitos en el siguiente modulo!
Buenas Mati! Estuve viendo por un poco el codigo y hay algunos cambios que estarian buenos hacer.
Tenes llamadas a los models desde los endpoitns, estaria bueno que el controller este de intermediario para eso, es decir que el flujo seria -> endpoint / controller / models y no endpoints / models.
Vi que en algunos endpoints le mandas al controller el req, trata de que siempre sea lo justo y necesario. Un caso esta en /api/search, en el getOffsetAndLimit le mandas todo el req que puede tener un monton de cosas.
Te recomiendo crear una lib "sendgrid" y ahi centralizar todo lo que sea mails para que sea mas escalable
Esta bueno que hayas tipado el model order, suma como documentacion y comprender que info y que tipo tiene esa collection.
Me llama la atencion el uso de airtable cuando creas una orden y como lo modificas en "listenMerchantOrder". No conviene en este ultimo caso hacer el update solo en firebase en vez de firebase y airtable?
En cuanto pueda ojeo los endpoints en postman para ver como va devolviendo la data.