Skip to content

Movies DB - #70

Open
0dayIg0r wants to merge 7 commits into
masterfrom
rodada-cases-semana3
Open

Movies DB#70
0dayIg0r wants to merge 7 commits into
masterfrom
rodada-cases-semana3

Conversation

@0dayIg0r

@0dayIg0r 0dayIg0r commented Apr 9, 2022

Copy link
Copy Markdown
Collaborator

No description provided.

@labenu-bot labenu-bot left a comment

Copy link
Copy Markdown

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
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

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