Skip to content

Projeto lab eddit#37

Open
ElisonMoises wants to merge 7 commits intomasterfrom
projeto-LabEddit
Open

Projeto lab eddit#37
ElisonMoises wants to merge 7 commits intomasterfrom
projeto-LabEddit

Conversation

@ElisonMoises
Copy link
Copy Markdown
Collaborator

@ElisonMoises ElisonMoises commented Jul 25, 2022

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

http://youthful-hole.surge.sh/

Imagens

TIRE PRINTS DAS TELAS DO SEU SITE E COLE AQUI
Capturar LabEddit

labenu-bot
labenu-bot previously approved these changes Aug 3, 2022
Copy link
Copy Markdown

@labenu-bot labenu-bot left a comment

Choose a reason for hiding this comment

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

Parabéns pela entrega!

Requisitos do projeto ✅

Implementações Feito
É possível fazer cadastro na aplicação
É possível fazer login na aplicação
Ao fazer login, o usuário é redirecionado para a página de feed de posts
Ao se cadastrar, o usuário também é redirecionado para a página de feed de posts
Há uma página de posts contendo todos os posts da aplicação
É possível criar um novo post através de um formulário
Cada post possui: nome de usuário que postou, o texto do post, o número de votos (positivo ou negativo) e o número de comentários -
É possível votar positiva ou negativamente no post e o número de votos é atualizado
Quando o usuário clica em um post, ele é redirecionado para a página de detalhe daquele post
Na página de detalhe de um post, são mostradas as informações do usuário, texto, curtidas e número de comentários
É possível criar comentários através de um formulário
Cada comentário é exibido e possui: o nome do usuário, o texto do comentário e o número de votos (positivos e negativos)
As páginas de feed ou de posts não devem ser acessadas por um usuário que não esteja logado
Quão fácil foi achar as informações no site? Médio
O que você achou do design do site? Bom
[Desafio] Todas as páginas são responsivas
[Desafio] É possível realizar Logout
[Desafio] É possível 'Compartilhar', que permite mandar o post para outra rede social -
[Desafio] É possível buscar por nome/texto do post no feed -
[Desafio] Há loading em todas as páginas que esperam dados da API (lista de posts, detalhe de post, criação de post)
Funcionalidade extra -
Feedback do código Feito
Todos os erros são tratados e exibidos adequadamente para o usuário -
Isolou corretamente as constantes do axiosConfig
Requisições não estão dentro dos componentes visuais
Quando um novo post/comentário é criado, ele aparece na tela sem a necessidade de atualizá-la
Não 'forçou' a atualização com o método reload()
Não causou um loop infinito nas requisições chamadas no useEffect
Removeu todos os console.logs -
Boa indentação
Criou link do surge
Criou um bom readme com informações relevantes para o projeto -

Comentários da pessoa avaliadora

Olá Elison!

O seu Labeddit ficou bastante completo e funcional, você também organizou muito bem o seu código e arquivos, parabéns!

Sobre o layout, você acabou esquecendo ou mesmo não adicionando o texto do post no feed, isso acaba prejudicando a usabilidade do site, pois o usuário precisa abrir um post para então ver do que ele se trata.

Um alerta para que lembres de tirar todos os console.log do código, para que fique limpo e nada apareça ao usuário. No momento não estamos trabalhando com dados sensíveis, mas ao ponto em que estivermos com usuários reais, isso pode ser um problema para a segurança dos dados. O melhor é receber os dados e erros da API e então tratá-los para serem exibidos (os que são necessários) ao usuário. Portanto, sempre que tiver um console.log "perdido" pelo código, verifique se são infos que precisas passar para o usuário e, caso sim, veja se o usuário já pode recebê-las, então apague ! (se não são infos para o usuário "e elas já foram usadas da maneira que precisavas, apaga também). Assim você consegue ter um controle do que o usuário pode acessar e deixa seu código bem limpinho.

Faltou também trazer um readme com informações sobre o seu projeto como tecnologias utilizadas (react, js, styled-components, axios, etc.), como usar o site (coisas bem óbvias mesmo, quais botões levam pra onde, como acessar informações, etc... um manual de instrução mesmo) e o link de acesso. Assim qualquer pessoa, dev ou não, que quiser ver teu projeto, vai ter acesso a essas informações.

No mais, parabéns pela sua entrega!

Tenha uma excelente semana!!!

Análise realizada por: Clara

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