Skip to content

Conversation

@caioteixeiraa
Copy link

Oi grupo 13!

Esse é um PR de correção, então não precisam mergeá-lo :D

O projeto de vocês ficou bem legal e com algumas funcionalidades a mais do que pedimos! Parabéns!!

Alguns comentários sobre o código de vocês:

  • Tem bastante código comentado, é uma boa prática evitar subir eles no PR.
  • Um problema no filtro de vocês é que quando coloca valor máximo 300, por exemplo, o produto de 300 reais não aparece D: Mas para consertar isso é bem simples :D

Parabéns pelo trabalho de vocês e continuem assim! 👏 👏

loures90 and others added 30 commits January 6, 2021 13:51
Feito componentes e lógica da seção de produtos.
…falta fazer o contador funcionar. Alterado display da seção carrinho e filtro. Alterado estilo da seção filtro também.
Merge pull request #5 from future4code/master
Resolvido bug na visualização do carrinho.
atualização de minha branch
home e carrinho estão prontos- falta o filter
Projeto final, com todas as funionalidades.
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.

5 participants