You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hemos realizado la revisión de tu repositorio y hemos encontrado los siguientes puntos de mejora :)
Si tienes alguna inquietud o deseas solicitar una nueva revisión puedes comunicarte con Ana Belisa por Slack y enviar por el formulario nuevamente tu repositorio.
El proyecto tiene un gran rendimiento, así como una interfaz amigable, al punto que (confieso), sin leer la documentación, pude entender de qué iba el proyecto, lo cual es excelente. En cuanto al código, se puede observar ciertas áreas de mejora:
La función cancelOrder (así como otras), podría reducirse usando funciones auxiliares, en la que se pasa la lista de objetos a las que se les aplicará el remove o el add. En pseudocódigo:
Por objeto en lista:
objeto.remove(“hidden”)
De igual forma, el string que tiene el prepPanel (líneas 36 – 41) es casi idéntico como en la función startPrep (líneas 105 – 110). Podría volverse un string formateado, en el que solo se pasa el texto del
. Ya sea “Preparing” o “Serving”.
Para el caso de los setTimeout (líneas 93-100), se ve que quieres que ocurra algo después de otra cosa, considero que podrías hacer uso de async await para el mejor manejo de encadenamientos, y así garantizar que se ejecutan en orden, sin tener que calcular los tiempos.
The text was updated successfully, but these errors were encountered:
Hemos realizado la revisión de tu repositorio y hemos encontrado los siguientes puntos de mejora :)
Si tienes alguna inquietud o deseas solicitar una nueva revisión puedes comunicarte con Ana Belisa por Slack y enviar por el formulario nuevamente tu repositorio.
El proyecto tiene un gran rendimiento, así como una interfaz amigable, al punto que (confieso), sin leer la documentación, pude entender de qué iba el proyecto, lo cual es excelente. En cuanto al código, se puede observar ciertas áreas de mejora:
De igual forma, el string que tiene el prepPanel (líneas 36 – 41) es casi idéntico como en la función startPrep (líneas 105 – 110). Podría volverse un string formateado, en el que solo se pasa el texto del
. Ya sea “Preparing” o “Serving”.
Para el caso de los setTimeout (líneas 93-100), se ve que quieres que ocurra algo después de otra cosa, considero que podrías hacer uso de async await para el mejor manejo de encadenamientos, y así garantizar que se ejecutan en orden, sin tener que calcular los tiempos.
The text was updated successfully, but these errors were encountered: