Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

End Proyect squad Mariana & Mariel #14

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

MarianaOlimpia
Copy link

No description provided.

Copy link

@reloadercf reloadercf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mariel y Olimpia, me gusto mucho su proyecto y sobre todo el diseño, lo cual es importante resaltarlo, tengo 2 recomendaciones: 1) Tengan cuidado con el uso de 3 maneras de generar estilos porque parecía que se instalaban dos frameworks de diseño y adicional sass, si no van a usar bootstrap entonces borrenlo.
2) Veo componentes super GIGANTES lo cual pueden mejorar modularizando mejor sus componentes.
Su proyecto es muy bueno y en general esta muy genial.

@@ -0,0 +1,26 @@
# See https://help.github.com/articles/ignoring-files/ for more about ignoring files.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

esto esta genial, recuerda siempre tener tu gitignore configurado

"last 1 chrome version",
"last 1 firefox version",
"last 1 safari version"
]

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

recuerda que puedes tener dependencias en modo dev

Learn how to configure a non-root public URL by running `npm run build`.
-->
<title>Burger Queen</title>
<link rel="stylesheet" href="https://stackpath.bootstrapcdn.com/bootstrap/4.5.0/css/bootstrap.min.css" integrity="sha384-9aIt2nRpC12Uk9gS9baDl411NQApFmC26EwAOH8WgZl5MYYxFfc+NcPb1dKGj7Sk" crossorigin="anonymous">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Si no vaz a usa bootstrapcdn entonces ten cuidado y borralo porque podría tener conflictos.

Comment on lines +13 to +32
<Router>
<Switch>
<Fragment>
<Fragment>
<Route path ='/' exact component = {IndexApp} />
</Fragment>
<Fragment>
<Fragment>
<Route path ='/orden' component={OrderSection}/>
</Fragment>
<Fragment>
<Route path= '/cocina' component= {KitchenSection}/>
</Fragment>
<Fragment>
<Route path = '/historial' component = {HistorialSection}/>
</Fragment>
</Fragment>
</Fragment>
</Switch>
</Router>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esto aun lo puedes meter en un componente

Comment on lines +1 to +4
$bqcream: #FCFAE3;
$bqpink: #FF7BAC;
$bqgreen: #1EBC9E;
$btMenu: '../src/Assets/btnMenu.png';

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
$bqcream: #FCFAE3;
$bqpink: #FF7BAC;
$bqgreen: #1EBC9E;
$btMenu: '../src/Assets/btnMenu.png';
$btMenu: '../src/Assets/btnMenu.png';
$bqcream: #FCFAE3;
$bqpink: #FF7BAC;
$bqgreen: #1EBC9E;

Comment on lines +35 to +37
render: (id) =>{
return <DeleteTwoTone onClick={() => deleteProducto(id)} />
},

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esto esta cool

Suggested change
render: (id) =>{
return <DeleteTwoTone onClick={() => deleteProducto(id)} />
},
render: (id) =>{
return <DeleteTwoTone onClick={() => deleteProducto(id)} />
},

)
} else {
return (
<div><Select placeholder="Mesa" style={{ width: 175 }} onChange={handleChange} value={carrito.mesa}>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dato adicional

Suggested change
<div><Select placeholder="Mesa" style={{ width: 175 }} onChange={handleChange} value={carrito.mesa}>
<div><Select placeholder="Mesa" style={{ width: 175 }} onChange={()=>{handleChange() secondHandle()}} value={carrito.mesa}>

Comment on lines +9 to +17
let [carrito,setCarrito]=useState({
nombreCliente:"",
total:0,
item:[],
status:false,
pay:false,
mesa:"",
orden: (""+Math.random()).substring(2,7),
mesero: ""

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no español

Suggested change
let [carrito,setCarrito]=useState({
nombreCliente:"",
total:0,
item:[],
status:false,
pay:false,
mesa:"",
orden: (""+Math.random()).substring(2,7),
mesero: ""
let [carrito,setCarrito]=useState({
nombreCliente:"",
total:0,
item:[],
status:false,
pay:false,
mesa:"",
orden: (""+Math.random()).substring(2,7),
mesero: ""

Comment on lines +1 to +4
import React from 'react';
import '../NavHeader/NavHeader.scss'
import Button from '../../Button/Button';
import { useHistory } from "react-router-dom";

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esto esta raro de leer

Suggested change
import React from 'react';
import '../NavHeader/NavHeader.scss'
import Button from '../../Button/Button';
import { useHistory } from "react-router-dom";
import React from 'react';
import { useHistory } from "react-router-dom";
import '../NavHeader/NavHeader.scss'
import Button from '../../Button/Button';

@@ -0,0 +1,26 @@
@import '~antd/dist/antd.css';

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sugerencia si vaz usar fuente utiliza google Font, le dará un buen estilo a tu frontend

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants