Skip to content

Natal labeddit#39

Open
Matheus0351996 wants to merge 5 commits intomasterfrom
natal-labeddit
Open

Natal labeddit#39
Matheus0351996 wants to merge 5 commits intomasterfrom
natal-labeddit

Conversation

@Matheus0351996
Copy link
Collaborator

@Matheus0351996 Matheus0351996 commented May 9, 2022

POR FAVOR, EDITE ESSA MENSAGEM INSERINDO AS INFORMAÇÕES DO SEU PROJETO. COMECE APAGANDO ESSA LINHA.

Labeddit

O que funciona

Botões, login, cadastrar.

O que não funciona

  • DESCREVA O QUE NÃO FUNCIONA NO SEU PROJETO

Link Surge

natall-labeddit.surge.sh

Imagens

TIRE PRINTS DAS TELAS DO SEU SITE E COLE AQUI

Copy link

@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? Difícil
O que você achou do design do site? Médio
[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

Matheus, faltaram muitas coisas no teu projeto e as implementações que você fez poderiam estar melhores. Sugiro que você assista a resolução do projeto e refaça. Estamos em uma etapa decisiva do processo de aprendizado, então é importante que você fixe com qualidade o que vimos até aqui.

Análise realizada por: Vitor Lopes

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.

2 participants