Avaliação Desafio Tagchatter - ReactJS

[bartier]

Olá pessoal,

Gostaria de indicar meu repositório para review. Tagchatter.

O código já faz o que foi requisitado pelo desafio, porém gostaria de saber o que poderia melhorar (boas práticas)...

Muito obrigado desde já!

[edgarberlinck]

Oi amigo, abra um pull request para uma outra branch. Fica mais fácil para postar os comentários.

[edgarberlinck]

Vou colocar algumar considerações aqui:

Button/index.js

Este componente deve ser declarado como um componente burro (Presentational ou Dumb component). Você já fez isso no ParrotsCounter e SideBar. : https://medium.com/@thejasonfile/dumb-components-and-smart-components-e7b33a698d43

Header/index.js Mesma observação.

TagCharter.js Logo no construtor você tem um setInterval responsável por recarregar as mensagens. Acho legal um clearInterval quando você destruir o componente (componentWillUnmount)

Mais interessante ainda seria se a responsabilidade de recarregar as mensagens fosse do componente Messages.

As chamadas as API poderiam estar em mais alto nível. Você poderia ter por exemplo:

• No lugar de api.get('/messages') prefira api.listarMensagens() ou api.listMessages()
• Mesma coisa para os métodos:
  • api.get("/messages/parrots-count")
  • api.post('/messages', newMessage),
  • api.put(/messages/${message.id}/unparrot, {})
  • api.put(/messages/${message.id}/parrot, {})

[edgarberlinck]

Ah, um componente react possui um método chamado componenetDidCatch que é muito útil para tratar exeptions.

[edgarberlinck]

Lembre-se de fechar esta issue quando estiver satisfeito com os reviews. Ok?

[bartier]

Oi amigo, abra um pull request para uma outra branch. Fica mais fácil para postar os comentários.

seria isso que você falou para fazer?

edit: vi que criou conflitos com esse PR...

[edgarberlinck]

Sim. Vou dar uma olhada na segunda feira.

[bartier]

@edgarberlinck obrigado!

[edgarberlinck]

@bartier As considerações feitas até o momento estão ok? Gostaria de mais alguma revisão ou já está satisfeito? Se sim, favor fechar esta issue.

[bartier]

@edgarberlinck As considerações estão ok sim, muito obrigado! Fecharei a issue.