Conversation
labenu-bot
left a comment
There was a problem hiding this comment.
Parabéns pela entrega!
Requisitos do projeto ✅
| Implementações | Feito |
|---|---|
| O usuário deve ter acesso a uma listagem dos filmes mais populares do dia | ✅ |
| O usuário deve conseguir paginar a lista para encontrar novos filmes | - |
| O usuário deve conseguir filtrar os filmes listados por gênero, com a possibilidade de usar mais de um gênero | - |
| O filtro deve ser persistido pela paginação | - |
| O usuário deve conseguir remover filtros e a listagem deve ser atualizada de acordo com o filtro removido | - |
| O usário deve ter acesso a uma outra página com detalhes sobre o filme, ao clicar em um item na listagem | ✅ |
| O usuário deve conseguir voltar para a página de listagem de filmes com os filtros ainda ativos | - |
| O app foi hospedado em um serviço de cloud(exemplo: Surge, Netlify, ZEIT ou AWS) | ✅ |
| O app foi criado usando React | ✅ |
| Fez README.md com as instruções para construir seu projeto localmente | ✅ |
| O app se comporta da mesma forma na última versão estável dos seguintes browsers: Chrome, Firefox, Edge | ✅ |
| A página com detalhes de um filme possui uma rota própria e está preparada para ser indexada em mecanismos de pesquisa | ✅ |
| O app é responsivo | - |
| Usar um framework para server-side rendering (SSR) ou static generation (SSG) como: Next.js, Gatsby, Nuxt.js. | - |
| Aplicar uma solução CSS-in-JS(como styled-components) ou CSS Modules | ✅ |
| Testes | - |
| Funcionalidade extra | ✅ |
| Feedback do código | Feito |
|---|---|
| Isolou corretamente as constantes do `axiosConfig | ✅ |
| Todos os erros são tratados e exibidos adequadamente para o usuário | - |
Não causou um loop infinito nas requisições chamadas no useEffect |
✅ |
Comentários da pessoa avaliadora
Olá Natalia!
Primeiramente, parabéns demais pela entrega deste case e por sua dedicação no curso, principalmente durante as últimas semanas!
Sobre o layout e usabilidade: eu gostei muito do efeito que você trouxe no hover das imagens, esse efeito ajuda a mostrar para o usuário que existe uma interação com esse card e mostra mais um pouco de conhecimento teu! Outro detalhe que ficou excelente foi trazer o link da home no Logo, ajuda o usuário a retornar a página e melhora demais a usabilidade do site! Parabéns!!!
Este layout é simples, o que torna fácil de adaptar para a versão mobile. Como vem crescendo cada vez mais o uso mobile em comparação com desktop, recomendo fazer a adaptação, ainda que de forma simplificada, pois é bastante visado no mercado. Um detalhe simples que pode ajudar é usar o display flex juntamente com o flex-wrap, para que a disposição dos cards se adeque ao tamanho da tela do usuário.
Você construiu um bom readme para o projeto, mas acho que faltou trazer também o link do site no surge, para que possa ser facilmente acessado, além da lista de tecnologias utilizadas para a criação do site. Assim você mostra que sabe e como fez. Se conseguir, é legal trazer, no readme, um "como usar" do site, bem explicado, falando como acessar um ou outro ponto dele e também como rodar o projeto localmente.
Sobre o teu código, você pode dar uma limpada nele, removendo linhas em branco desnecessárias, trechos comentados do código, console.log que ficaram ainda (depois de avaliar, nesses logs o que ainda falta ser exibido ao usuário, como os erros e tratá-los para que sejam corretamente exibidos) e arrumando alguns detalhes da indentação.
No mais, parabéns pela conclusão do curso!
Análise realizada por: Clara
PROMOBIT
Usando a API de filmes gratuita themoviedb em sua versão 3, foi criado uma listagem dos filmes mais populares do dia, consultando o endpoint GET /movie/popular para realizar a listagem.
Ao clicar em um item dessa listagem, outra página com os detalhes do filme escolhido é exibida. Foiusado o endpoint GET /movie/{movie_id}.
O que funciona
O que não funciona
Link Surge
boorish-chair.surge.sh
Imagens