Skip to content

PR de Correção#12

Merged
ricardo-ramos merged 23 commits intocorrecao-projetofrom
master
Feb 4, 2021
Merged

PR de Correção#12
ricardo-ramos merged 23 commits intocorrecao-projetofrom
master

Conversation

@leticia-chijo
Copy link

PR de Correção

ricardo-ramos and others added 22 commits January 17, 2021 22:39
Filtro
Filtro estilizado dentro do App
…l, com um header ou um dropdown clicável, que não consegui no momento
carrinho incluído, mas não é possível adicionar produtos, pois dá erro
carrinho funcionando
correção do package.lock e cursor: pointer em links
notificação ao clicar em adicionar ao carrinho
Copy link
Author

@leticia-chijo leticia-chijo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oi gente! O projeto de vocês foi avaliado como dentro do esperado, parabéns! 🎉

O site está funcional e o código de vocês está organizado. Faltou implementar a parte de filtros e ordenação dos itens na home, mas acredito que com um pouco mais de tempo vocês teriam conseguido!

Um probleminha que reparei é que quando eu adiciono dois itens no carrinho, altero a quantidade de um deles e deleto o outro, a quantidade que eu coloquei no item não deletado volta a ser 1. Um pequeno bugzinho apenas hehe.

Outra coisa que vocês poderiam fazer para deixar o projeto ainda melhor seria editar o arquivo Readme! Coloquem os inegrantes do grupo, o que funciona ou não e, principalmente, o link do surge do projeto de vocês.

Mas o site de vocês está bem legal, gostei da notificação ao adicionar novos itens no carrinho 😄

Se ficou alguma dúvida de como fazer algo no projeto, principalmente da parte de filtros, vocês podem assistir aos vídeos do Thinking in React e, em breve, soltarei um documento com os erros mais comuns e suas soluções.

É isso, agora vamos focar nos próximos projetos que ainda tem bastante coisa pela frente!

@ricardo-ramos ricardo-ramos merged commit fae660e into correcao-projeto Feb 4, 2021
@ricardo-ramos
Copy link
Collaborator

Oi gente! O projeto de vocês foi avaliado como dentro do esperado, parabéns!

O site está funcional e o código de vocês está organizado. Faltou implementar a parte de filtros e ordenação dos itens na home, mas acredito que com um pouco mais de tempo vocês teriam conseguido!

Um probleminha que reparei é que quando eu adiciono dois itens no carrinho, altero a quantidade de um deles e deleto o outro, a quantidade que eu coloquei no item não deletado volta a ser 1. Um pequeno bugzinho apenas hehe.

Outra coisa que vocês poderiam fazer para deixar o projeto ainda melhor seria editar o arquivo Readme! Coloquem os inegrantes do grupo, o que funciona ou não e, principalmente, o link do surge do projeto de vocês.

Mas o site de vocês está bem legal, gostei da notificação ao adicionar novos itens no carrinho

Se ficou alguma dúvida de como fazer algo no projeto, principalmente da parte de filtros, vocês podem assistir aos vídeos do Thinking in React e, em breve, soltarei um documento com os erros mais comuns e suas soluções.

É isso, agora vamos focar nos próximos projetos que ainda tem bastante coisa pela frente!

Muito obrigado pelo feedback, Letícia! 😄 Não tinha reparado nesse bug do carrinho, ops haahahahahah... Sobre editar o readme, é uma excelente ideia. Ainda estamos nos habituando devidamente ao Github (eu até tinha uma ideia beeeeeeem vaga por ter mexido em 2012, mas esqueci praticamente tudo e isso só ajudou a não me perder tanto nas aulas relacionadas a ele, mas foi praticamente um novo aprendizado).

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.

3 participants

Comments