preparando para o review #1
Conversation
There was a problem hiding this comment.
Olá, Diego!
Aqui embaixo tem algumas recomendações quanto ao seu projeto, mas além disso, gostaria de pontuar algumas coisas:
- Você, talvez por pouco tempo na hora de organizar o projeto, não importou (no backend) os schemas no lugar certo. Por ser uma linha de código, não desconsiderei o projeto todo por isso, apenas fui lá e troquei.
- Seria uma boa você redirecionar o usuário para a home após ele adicionar alguma transação, tornando sua aplicação mais fluída.
Tirando isso, sua aplicação está ótima!
Abraços 😷
| function totalCalculation () { | ||
| let total = 0; | ||
| box.map(item => { | ||
| total+= Number(item.box); | ||
| }) | ||
| return (total) | ||
| } |
There was a problem hiding this comment.
E se isso viesse do backend como
{
balance: value,
transactions: []
}
| useEffect(() => { | ||
| if(!user) { | ||
| history.push("/") | ||
| return("") |
There was a problem hiding this comment.
Seu return pode ser um return vazio:
return;
| const body = { | ||
| transaction: value, | ||
| description: description | ||
| } |
There was a problem hiding this comment.
E se isso já estivesse no useState como um objeto?
| event.preventDefault(); | ||
|
|
||
| if(password !== passwordConfirm) { | ||
| alert("The confirmation password must be the same as ur password"); |
There was a problem hiding this comment.
É uma boa prática não colocar os erros como alertas, assim sua aplicação fica mais fluída com erros no próprio corpo da aplicação.
| import { newTransaction } from "../service"; | ||
| import UserContext from "../contexts/UserContext"; | ||
|
|
||
| export default function Spending () { |
There was a problem hiding this comment.
Esse componente não é similar ao Profit? Como poderíamos torná-lo um componente único?
No description provided.