finish project#18
Conversation
code review ok, the integration is fine
reloadercf
left a comment
reloadercf
left a comment
There was a problem hiding this comment.
Jessy, Shei & Abi, me agrada la logica seguida para resolver su proyecto, tengo recomendaciones:
1)Consistencia, algunas variable estaban en español y otras en ingles, NO USAR camelcase en css y ten cuidado con el formato (identaciones y todas las importaciones)
2)pudiste reutilizar algos elementos que vi repetitivos, por otro lado había archivos que superaban las 200 Linas de código, ten cuidado con esto porque puede seguir creciendo el código y resulte confuso en un futuro
3)tu código tuvo muchas lineas que no se utilizan lo cual te recomiendo BORRAR
4) El uso de GitHub para seguir la linea de tiempo de su código (solo veo cambios de 2 personas) CUIDADO.
Lo que más me gusto fue el uso de firebase y las funciones que utilizaron para actualizar y guardar en firebase.
| @@ -0,0 +1,65 @@ | |||
| # Logs | |||
| @@ -1,4 +1,4 @@ | |||
| # Burger Queen | |||
| # Burger Queen Laboratoria | |||
There was a problem hiding this comment.
complementar el readme con: diagramas de flujo o capturas de pantalla y manual de usuario
| @@ -0,0 +1,23 @@ | |||
| # See https://help.github.com/articles/ignoring-files/ for more about ignoring files. | |||
|
|
|||
There was a problem hiding this comment.
tienen 2 gitignore lo cual ESTA MAL
| @@ -0,0 +1,38 @@ | |||
| { | |||
| "name": "my-app", | |||
There was a problem hiding this comment.
| "name": "my-app", | |
| "name": "Burger Queen", |
| work correctly both with client-side routing and a non-root public URL. | ||
| Learn how to configure a non-root public URL by running `npm run build`. | ||
| --> | ||
| <title>React App</title> |
There was a problem hiding this comment.
| <title>React App</title> | |
| <title>Burger Queen</title> |
| <Switch> | ||
| <Route path="/Comida"> | ||
| <Comida ProductsFo={ProductsFo} addItemToOrder={addItemToOrder} handleCounter={handleCounter} /> | ||
| </Route> | ||
| <Route path="/"> | ||
| <Desayuno ProductsBF={ProductsBF} addItemToOrder={addItemToOrder} handleCounter={handleCounter}/> | ||
| </Route> | ||
| </Switch> |
There was a problem hiding this comment.
debes usar link to
en vez de crear un nuevo router
| </tr> | ||
|
|
||
| </table> | ||
| <button onClick = { () => {sendOrderKitchen({ |
| //agregar estilos | ||
| //imprimir precio |
There was a problem hiding this comment.
| //agregar estilos | |
| //imprimir precio |
| <button | ||
|
|
||
| key={product.id} | ||
| onClick={()=>addItemToOrder(product)} | ||
| > | ||
| <span>{product.nameProduct} {product.price}</span> | ||
| </button> |
| @@ -0,0 +1,22 @@ | |||
| import {v4 as uuidv4} from "uuid"; | |||
|
|
|||
There was a problem hiding this comment.
el próximo reto es que consumas esta data desde firebase
4 participants
No description provided.