PR de correção. Não precisa mergear#8
PR de correção. Não precisa mergear#8pedro-severo wants to merge 24 commits intocorrecao-projetofrom
Conversation
layout produtos pronto
finalizando home
segue outra parte do código.
Atualizando filtro 2
Projeto Labestore
| /* useEffect(() => { | ||
| const data = items.map((product) => ({ | ||
| ...product, | ||
| priceFormatted: formatPrice(product.price), | ||
| })); | ||
| setItems(data); | ||
| // eslint-disable-next-line react-hooks/exhaustive-deps | ||
| }, []); */ |
There was a problem hiding this comment.
Lembrem de tirar os comentários de códigos não usados.
|
|
||
| import { formatPrice } from "../../util/format"; | ||
|
|
||
| import { Container, ProductTable, Total } from "./styles"; |
There was a problem hiding this comment.
Perfeito. Esse isolamento de constantes do styled é uma excelente prática.
| console.log(props.valor); | ||
| }; | ||
| const comparaProduto = (event) => { | ||
| console.log(event.target.value); |
There was a problem hiding this comment.
Lembrem de tirar os console.logs usados.
| const productIndex = item.findIndex((p) => p.id === id); | ||
| if (productIndex !== -1) { | ||
| item.splice(productIndex, 1); | ||
| toast.error("Item removido do carrinho 😢 "); |
There was a problem hiding this comment.
Hahahaha ótimo emoji para descrever alguém tirando um produto do seu carrinho
| })} | ||
| </ProductList> | ||
| </> | ||
| ); |
There was a problem hiding this comment.
216 linha de componente. Outros componentes também ficaram grandes. Semprem tirem um tempo para componentizar o projeto.
|
Querida(o)s, fiz alguns comentários ao longo do código... deem uma olhada. Além disso, o Darvas disponibilizou ontem (02 de junho) o gabarito do exercício em vídeo (thinking in react). Deem uma olhada lá também. Parabéns pela entrega! Na próxima, não se esqueçam do link do surge. |
pedro-severo
requested review from
AlexandreGessone,
Paulopma,
atmirala and
jonatan-machado
3 participants
No description provided.