Skip to content

promobit - #53

Open
ghost wants to merge 7 commits into
masterfrom
promobit
Open

promobit#53
ghost wants to merge 7 commits into
masterfrom
promobit

Conversation

@ghost

@ghost ghost commented Apr 5, 2022

Copy link
Copy Markdown

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

  • renderização da lista de filmes populares do dia
  • detalhes sobre o filme selecionado
  • botão home na logo

O que não funciona

  • renderizar mais detalhes do filme selecionado
  • paginação
  • filtros
  • recomendações
  • renderização do elenco

Link Surge

boorish-chair.surge.sh

Imagens

image

image

image

image

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

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.

1 participant