Skip to content

PR de correção #12

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
IsraelMCordeiro wants to merge 31 commits into
base: correcao-projeto
Choose a base branch
from
Open

PR de correção #12

IsraelMCordeiro wants to merge 31 commits into from

Conversation

@IsraelMCordeiro
Copy link

@IsraelMCordeiro IsraelMCordeiro commented Jul 30, 2022

não mergear!!!

VieiraMeloRoberta and others added 30 commits July 17, 2022 18:19
@VieiraMeloRoberta
em andamento
5129a0d
@VieiraMeloRoberta
Merge pull request #1 from future4code/roberta-ecommerce
0f48c76 
em andamento
@Guilherme-Azevedo-Melim
Alteração feita nas importações
95ea138
@VieiraMeloRoberta
Estilização
6351e81
@sterx17
Merge branch 'master' into roberta-ecommerce
2a8c7b8
@VieiraMeloRoberta
Merge pull request #2 from future4code/roberta-ecommerce
5e2a857 
Estilização
@sterx17
<feat>: inclusão da feature Filtro ao projeto
801e19f
@Guilherme-Azevedo-Melim
Estilização separada por components
76ef052
@Guilherme-Azevedo-Melim
Merge pull request #3 from future4code/Stephany-Filtro
bd6a245 
<feat>: inclusão da feature Filtro ao projeto
@sterx17
<conflict>: correção de conflitos na branch guilhermer-carrinho
4343d64
@sterx17
Merge pull request #4 from future4code/guilhermer-carrinho
f8017f6 
Estilização separada por components
@sterx17
testando commits na nova maquina
319fae2
@sterx17
Merge pull request #5 from future4code/Stephany-Filtro
c03e788 
testando commits na nova maquina
@sterx17
<style>:ajustes visuais nos cards
4532e53
@sterx17
<style>: ajustes de estilos do card e modificação do mock de dados
bbfdbab
@sterx17
<surge> inclusão do link surge para a entrega
cb691d9
@sterx17
Merge pull request #7 from future4code/Stephany-Filtro
a9ac36d 
<surge> inclusão do link surge para a entrega
@Guilherme-Azevedo-Melim
ordenação feita
e5a8c87
@sterx17
<feat>: inclusão do localStorage/useEffect no carrinho
08341a1
@sterx17
<fix>: correção de comportamento dos filtros
fe89bcd
@Guilherme-Azevedo-Melim
Merge pull request #9 from future4code/Stephany-Filtro
24bf78e 
<fix>: correção de comportamento dos filtros
@Guilherme-Azevedo-Melim
Merge pull request #8 from future4code/Stephany-LocalStorage
cba86c6 
<feat>: inclusão do localStorage/useEffect no carrinho
@Guilherme-Azevedo-Melim
Merge pull request #6 from future4code/Stephany-EstiloCards
190b507 
Cards: estilos
@sterx17
<feat>: inclusão da ordenação da Home
7989f5d
@sterx17
<feat> inclusão da ordenação: incompleto. ajustes no header também
c1cbe81
@sterx17
Merge pull request #10 from future4code/Stephany-Ordenacao
63258e0 
<feat> inclusão da ordenação
@sterx17
Update README.md
fd9e374
@Guilherme-Azevedo-Melim
finalizado
c026acf
@sterx17
Merge branch 'master' into guilhermer-carrinho
6a3d8f5
@sterx17
Merge pull request #11 from future4code/guilhermer-carrinho
6a218a4 
Guilherme carrinho
@IsraelMCordeiro
Copy link
Author

IsraelMCordeiro commented Jul 30, 2022

Home e Lista de Produtos
Fui capaz de visualizar os produtos, preço de cada produto e a parte do carrinho
Carrinho
O carrinho funcionou perfeitamente, consegui aumentar e diminuir a quantidade e valor total era atualizado conforme as compras.
Filtros e Ordenação
Os filtros funcionaram, entretanto, a busca por nome só me permetiu procurar o nome examente como estava escrito no card.Não consegui pesquisar com letras maiúsculas ou minúsculas.

Comentários Gerais Sobre o Projeto
Parabéns, grupoooo!!!
Gostei bastante do layout, a home estava bem organizada e achei o máximo clicar no carrinho e ver outra tela só para ele.
Consegui realizar a compra, aumentar e diminuir a quantidade e fazer as pesquisas no filtro, mas, quando tentei pesquisar com letras maiúsculas ou minúsculas, não rolou. Tentem verificar isso com o includes.
No mais, estão voando grupo. Parabéns!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@IsraelMCordeiro @VieiraMeloRoberta @Guilherme-Azevedo-Melim @sterx17