Conversation
whats4 - desenvolvido por Rodrigo Souza e Artur Pelliciolo
amandarangel
left a comment
There was a problem hiding this comment.
Bom dia Artur e Rodrigo!
O projeto foi avaliado como: acima do esperado!
Parabéns pela entrega!!! O projeto ficou realmente muito bom, com as funcionalidades básicas de acordo com o esperado e com quase todos desafios feitos!
Só alguns pontos de atenção: 1) Olhem com mais carinho para a nomeação de variáveis e funções. Nomes abreviados como inputNomeF não são recomendados. Olhando o código de vocês eu sei que o F é de Form, mas em um código maior e mais complexo é mais difícil de subentender isso. Além disso, nome de funções como ckeck, enviar, apagar deixam um pouco a desejar no quesito que informação acerca do que essas funções realmente faze, Em um código maior poderia se rum problema. Então poderia ser: apagarMensagem, enviarMensagem, aoCLicarEnter. 3) O envio da mensagem ao clicar enter de vocês não está funcionando apenas porque vocês colocar o código da tecla Enter como 15 e é 13. 2) Procurem treinar o uso dos styled-components, pois é algo bem utilizado no mercado também. :)
| @@ -0,0 +1,70 @@ | |||
| # Getting Started with Create React App | |||
There was a problem hiding this comment.
Pessoal, uma coisa muito legal e comum no mercado é a utilização do README para colocar informações sobre o projeto. Então incentivamos vocês a editarem esse README padrão do React. O link do surge, inclusive, deve ficar aqui (falar nisso, não encontrei o link do surge na entrega de vocês)! :)
Esse é um PR de correção. Não é necessário mergear.