preparando para o review#1
preparando para o review#1sidneyouteiro wants to merge 1 commit intothipereira02:mainfrom sidneyouteiro:review
Conversation
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/thipereira02/trackit/6LR5iXitjnHmsoURr8i7YHTEemUv |
sidneyouteiro
left a comment
sidneyouteiro
left a comment
There was a problem hiding this comment.
Fala Thiago! Trouxe um feedback de código do seu último projeto 🙂
Eu gostei bastante do seu projeto. De maneira geral, mostrou domínio dos conceitos apresentados em aula e apresentou um código muito bem organizado, conciso e semântico.
Bom trabalho!
| } | ||
|
|
||
| function value(data){ | ||
| const done = data.filter(d => d.done===true); |
There was a problem hiding this comment.
Poderíamos reescrever esse trecho como:
const done = data.filter(d => d.done);
| <p>Sequência atual: <Sequence color={habit.done && color}>{habit.currentSequence} {habit.currentSequence===1 ? "dia" : "dias"}</Sequence></p> | ||
| <p>Seu recorde: <Sequence color={habit.highestSequence>0 && habit.highestSequence===habit.currentSequence}>{habit.highestSequence} {habit.highestSequence===1 ? "dia" : "dias"}</Sequence></p> |
There was a problem hiding this comment.
Poderíamos aproveitar algumas quebras de linha ou avaliar as operações booleanas por fora afim de aumentar a organização e legibilidade do código
1 participant
No description provided.