Skip to content

PR de correção#2

Open
amandarangel wants to merge 3 commits intocorrecaofrom
master
Open

PR de correção#2
amandarangel wants to merge 3 commits intocorrecaofrom
master

Conversation

@amandarangel
Copy link

Esse é um PR de correção. Não é necessário mergear.

Copy link
Author

@amandarangel amandarangel left a comment

Choose a reason for hiding this comment

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

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
Copy link
Author

Choose a reason for hiding this comment

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

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)! :)

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.

3 participants