Skip to content

PR de correção. Não precisa mergeear.#17

Open
pedro-severo wants to merge 75 commits intocorrecao-projetofrom
master
Open

PR de correção. Não precisa mergeear.#17
pedro-severo wants to merge 75 commits intocorrecao-projetofrom
master

Conversation

@pedro-severo
Copy link

ifabriciorodrigues and others added 22 commits June 12, 2020 19:08
…omprando, setando o estado cart para zero, ao finalizar compra
Atualização do carrinho, função adicionar parcelas, botão continuar c…
Comment on lines +57 to +58
const url =
"https://us-central1-labenu-apis.cloudfunctions.net/eloFourOne/products";
Copy link
Author

Choose a reason for hiding this comment

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

Boa galera! Muito boa prática isolar a url. Uma dica em cima disso: na próxima, salvem a url base em um doc.js isolado e exportem ele. Aí, qualquer componente vai poder ter acesso à constante, e vcs não vão precisar criar uma const url pra cada componente com requisição.

Comment on lines +137 to +168
if (this.state.category) {
filteredProducts = filteredProducts.filter(
(item) =>
item.category.toLowerCase() === this.state.category.toLowerCase()
);
}
if (this.state.minPrice) {
filteredProducts = filteredProducts.filter(
(item) => item.price >= this.state.minPrice
);
}
if (this.state.maxPrice) {
filteredProducts = filteredProducts.filter(
(item) => item.price <= this.state.maxPrice
);
}
if (this.state.searchInput) {
filteredProducts = filteredProducts.filter((item) =>
item.name.toLowerCase().includes(this.state.searchInput.toLowerCase())
);
}
if (this.state.sortOption === "nameAsc") {
filteredProducts.sort((a, b) =>
a.name.toLowerCase() > b.name.toLowerCase() ? 1 : -1
);
}
if (this.state.sortOption === "nameDec") {
filteredProducts.sort((a, b) =>
a.name.toLowerCase() > b.name.toLowerCase() ? -1 : 1
);
}
if (this.state.sortOption === "categoryAsc") {
Copy link
Author

Choose a reason for hiding this comment

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

Eitaaaa que tanto de if hahaha

@pedro-severo
Copy link
Author

Oi, queridoas!

Nossa, o trabalho de vcs ficou maravilhoso! Ficou um site bem bacana e com um uso muito bom do materialUI. Além disso, o código está bem escrito. Peço que vcs deem uma atenção para a componentização de alguns componentes bem grandes e para algumas lógicas, que podem ser "enxugadas".

Mas nada disso tira o bom trabalho feito. Vcs usaram muito bem todos os ensinamentos do módulo 2.

Parabéns! Continuem assim!

PS: adorei o nome da equipe - DevSerImpecável hahahahahaha

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.

6 participants