Movies DB - #70
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á Igor!
Parabéns por mais um case!!
O Layout desse app é simples, o que torna fácil de adaptar para a versão mobile e o seu já está quase adaptado, faltando poucos detalhes para que seja de fato responsivo. Como vem crescendo cada vez mais o uso mobile em comparação com desktop, recomendo fazer a adaptação, pois é bastante visado no mercado. Um detalhe simples que pode ajudar é colocar as imagens e tamanhos usando as unidades relativas, como vh/vw, que se adaptam ao tamanho da tela.
Na página de detalhes você fez uma barra de rolagem, tanto para exibir o elenco quanto sugestões, esse detalhe é muito importante para que essas informações não ocupem toda a tela do usuário e apresentam uma fluidez maior no conteúdo! Você também tomou o cuidado de estilizar a barra de rolagem e ficou ótimo! Parabéns!
Sobre seu código, ele está bastante organizado, mas falta ainda fazer uma limpeza, ficaram alguns comentários e console.log nele, os erros também estão em com um caminho um pouco diferente da resposta dada pela API, para acessá-los corretamente, você precisa colocar o e.response e ver qual o caminho a API te dá para acessar o erro de forma correta (na própria API tem essa informação).
No mais, o projeto está bastante bom, porém para apresentá-lo como portfólio, seria legal trazer mais informações no readme, falar mais do projeto, como funciona, qual a proposta, além de trazer apenas as instruções para rodar localmente e as tecnologias. Mostrando mais o seu domínio sobre ele enquanto desenvolvedor.
Parabéns pelo case e pela dedicação no curso!
Análise realizada por: Clara
No description provided.