PR Correção

[BrunoSilva28]

No description provided.

[labenu-bot]

Parabéns pela entrega!

Requisitos do projeto ✅

Implementações	Feito
A página possui dois inputs controlados: um para o nome do usuário e um para a mensagem	✅
Existe um botão de enviar e um espaço para as mensagens enviadas aparecerem	✅
Ao pressionar o botão de enviar, a mensagem enviada aparece na tela	✅
Ao pressionar o botão de enviar, o campo de mensagem é limpo	✅
Quão fácil foi achar as informações no site?	Fácil
O que você achou do design do site?	Médio
[Desafio] A mensagem pode ser enviada utilizando a tecla enter	✅
[Desafio] Quando o usuário é 'eu' ou 'Eu', a mensagem aparece do outro lado da tela	-
[Desafio] Ao clicar duas vezes em uma menasgem já enviada, ela é deletada	✅
Funcionalidade extra	-

Feedback do código	Feito
Criou componentes separados ao invés de deixar tudo no arquivo App.js	✅
Utilizou apenas styled-components para a estilização	-
Deu bons nomes para variáveis, funções, componentes e styled-components	-
Boa identação	-
Ao alterar arrays no estado, criou uma cópia da lista com o spread operator para alterá-la antes de utilizar o this.setState	✅
Criou link do surge	✅
Criou um bom readme com informações relevantes para o projeto	-

Comentários da pessoa avaliadora

Olá, pessoal! Primeiramente, gostaria de parabenizá-los por concluírem mais uma etapa do processo da Labenu!
Pude perceber que vocês deram conclusão em boa parte das funcionalidades, inclusive gostei da ideia de inserir imagens
para representar os usuários, ficou bem bacana! :) Sinto que estão no caminho certo!

De toda forma, gostaria de fazer alguns comentários que talvez ajudem vocês nos próximos projetos:

1. Vi que vocês deixaram em aberto alguns PR´s de modificação do código, apesar de estarem aprovados por membro da equipe. Seria interessante que, ao concluírem o projeto, não deixem nenhum deles em aberto, pois assim conseguimos identificar o que de fato faz parte do projeto final de vocês.

2. Notei que vocês precisaram criar um novo link de surge para o projeto final, diferente do de entrega. Seria interessante que vocês mantivessem o mesmo link, apenas modificando-o conforme necessário. Assim, conseguimos identificar exatamente qual é o link certo do projeto final de vocês. Caso tenham problemas em fazer essas modificações, vocês podem tentar também deletar o surge antigo e recriá-lo com o mesmo nome.

3. Notei que vocês se utilizaram bastante dos recursos de estilização nativa na configuração da página principal, a partir do arquivo App.css. Sei que às vezes pode dar bastante trabalho, mas é também uma ótima prática codar a estilização de forma padronizada (ou nativamente, ou usando styled-components, ou outra lib). Como sugestão, gostaria de encorajá-los a codar mais usando o styled-components, visto que este é um recurso interessante e que pode ajudar muito a entender o fluxo
   das estruturas.

4. Uma boa prática na codagem e nomeação de variáveis, funções e propriedades é seguir a regra do "camelCase" de escrita. Ou seja, atribuir nomes do tipo "nomeEmCamelCase", sendo a primeira letra da primeira palavra minúscula (não pode ser número ou símbolos), todas as palavras utilizadas na nomeação juntas(ou separadas por "_") e demais primeiras letras das palavras seguintes em maiúsculo. Pude perceber que na estruturação do código, algumas vezes vocês não seguiram tal prática (na nomeação das propriedades de estado, como "Conteudo" e "NomeRemetente", por exemplo). Sugiro que busquem ao máximo
   seguir esta formatação de escrita, pois futuros avaliadores e recrutadores avliarão em vocês a boa escrita de código.

5. Notei que algumas estruturas do código não seguiram totalmente a identação correta. Para auxiliá-los neste processo, caso utilizem o VSCode como ferramenta no Windows, vocês podem utilizar o atalho ALT+SHIFT+F, que faz a correção automática de identação do código. Para outros SO´s, sugiro a leitura desta página: "https://stackoverflow.com/questions/29973357/how-do-you-format-code-in-visual-studio-code-vscode"

6. Uma prática muito interessante é a de criar uma documentação README.md que seja visível a qualquer pessoa que eventualmente observe o projeto de vocês. Apesar terem pontuado informações sobre o que funciona ou não no código, seria
   interessante também que vocês incluíssem na documentação informações sobre a tecnologia utilizada ("Este projeto foi realizado utilizando React.js, ...), o propósito do projeto ("Este projeto busca implementar funcionalidades básicas do Whatsapp, ...) e, por se tratar de um projeto em equipe, também seria interessante identificar como foram realizadas as atividades da equipe (se pair programming, ou se não, como foram distribuídas as tarefas). Dessa forma, qualquer avaliador ou recrutador que eventualmente veja o portfólio futuro de vocês saberá mais ou menos a intenção do projeto, suas limitações e também como buscaram cooperar como equipe (esta, uma característica essencial nas empresas).

7. Notei que vocês conseguiram desenvolver muito bem os desafios. Todavia, na implementação de separação do usuário próprio ("eu" ou "EU"), a escrita à direita da tela só ocorre se passarmos em "Nome do Remente" o nome "eu" minúsculo. Como sugestão, vocês poderiam capturar a informação do input controlado e utilizar-se do método .toLowerCase() para tornar qualquer variação de escrita da palavra "eu" (Eu, eU, EU ou eu) em "eu" minúsculo (utilizar o método .toUpperCase() também funciona, mas de forma inversa).

8. Outra coisa que observei foi que vocês não bloquearam a exibição de mensagens ou usuários vazios. Como sugestão, sugiro
   a inclusão de condicionais na função de envio para que o state possa ser modificado somente quando os campos forem
   preenchidos.

9. Notei que ao inserir muitas mensagens no projeto de vocês, eventualmente a tela se arrasta para baixo, assim desconfigurando a estilização base do projeto. Como sugestão, gostaria de recomendar a vocês a leitura desta
   funcionalidade: "https://www.w3schools.com/howto/howto_css_custom_scrollbar.asp". Pode ser que ajude vocês em novos projetos! :)

No mais, parabéns pelo projeto! E muito sucesso na jornada de vocês! :)

Análise realizada por: Bruno