PR de Correção

[joaogolias]

POR FAVOR, EDITE ESSA MENSAGEM INSERINDO AS INFORMAÇÕES DO SEU PROJETO. COMECE APAGANDO ESSA LINHA.

INSIRA O NOME NOME DO PROJETO AQUI

O que funciona

• DESCREVA O QUE FUNCIONA NO SEU PROJETO

O que não funciona

• DESCREVA O QUE NÃO FUNCIONA NO SEU PROJETO

Link Surge

A PARTIR DA SEMANA 5, INSIRA AQUI O LINK DO SURGE. ANTES DISSO, APAGUE ESSAS DUAS LINHAS.

Imagens

TIRE PRINTS DAS TELAS DO SEU SITE E COLE AQUI

[joaogolias]

Pessoal, gostei muito do resultado de vocês e tenho algumas considerações para melhorar:

1. Antes de subir o projeto, lembrem-se de retirar todos os console.log. Isso é mal visto em alguns lugares porque, no final das contas, nós não queremos que o usuário consiga ver mensagens no dev tools quando tiver acesso ao projeto final.

2. Tentem colocar nomes melhores das variáveis. "val" e "xs" são um nome ruins, pois não indicam o que a variável faz

3. Quando usarem o find cuidado para sempre retornar algum valor booleano: true ou false

4. É melhor colocar as variáveis que correspondem ao styled components antes do componente principal do arquivo

5. Vocês aparentemente alteraram todo o package.json. Cuidado para isso não acontecer. Não consegui rodar o projeto de vocês por causa disso.

6. Na hora dos valores, vocês colocaram uma string assim: R${total},00. Isso não permite ter valores quebrados no site de vocês (R$10,50, R$100,90 e coisas assim).

7. Vocês criaram um array para representar as formas de pagamento. E os valores do value e do label eram os mesmos. Normalmente, quando fazemos assim, o value é algo diferente, sem espaços e sem acentos, para não ter problema com esses caracteres especiais (ex.: value poderia ser a_vista para o caso da label À vist

Por fim, gostaria de ressaltar que gostei muito que vocês utilizaram os alert para avisar o usuário quando ele fez algo errado! É muito importante coisas assim para UX. Além disso, fiquei bem contente que vocês utilizaram funções que não explicamos no curso propriamente, como find e reduce

Parabéns! Ótimo trabalho!