Skip to content

Conversation

@soter19
Copy link

@soter19 soter19 commented Mar 2, 2020

Esse PR é artificial, não precisa ser mergeado.

http://futureecommerce2.surge.sh/

marinaisabels and others added 30 commits February 20, 2020 09:49
"Foi criado a estrutura básica"
Botão faz o Carrinho Aparecer e Desaparecer
Componente Home - mostrando todos os produtos + produtos
No componente Home fizemos a Ordenação dos produtos e estilização
Finalizados requisitos obrigatórios + Readme.md
@soter19
Copy link
Author

soter19 commented Mar 2, 2020

Pontos de atenção:

  1. Muito boa a organização do código, sigam assim!!
  2. Lembrem-se de remover código comentado para a entrega.
  3. Cuidado com lógicas muito extensas dentro do método render, deem prioridade para extrair lógicas extensas para funções/métodos próprios, facilitando a leitura do código.
  4. Gostei dos comentários esclarecendo as lógicas que ocorrem no código, essa prática ajuda bastante a entender esse código no futuro quando vocês forem ler ele depois de muito tempo.
  5. Hehe, sei que foi corrido mas deixo como provocação um "Em projetos futuros gastem um tempinho com estilização"

@soter19 soter19 closed this Jul 1, 2024
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