search

devhatt / octopost

Octopost is an open source service from Devhat that was created to guide new developers for free
https://devhatt.github.io/octopost/
MIT License
65 stars 45 forks source link

chore: remove search clue component and its logic #474

Closed JpBurgarelli closed 2 months ago

JpBurgarelli commented 2 months ago

Closes #466

Feature Essa task tinha com objetivo remover o componen SearchClue. Basicamente, nesta Task, apenas removi o searchClue, suas importações e lógicas derivadas do sideBar onde era importado e fiz a covarage chegar a 100%. Porém, acabei tendo que remover também uma função/botão implementado pelo Zoldyck no sideBar, pois o ele estava ali, única e exclusivamente, para que, quando esse botão fosse clicado, as estilizações que ele fez para o sideBar na versão mobile fossem aplicadas/chamadas. Então, acabava se tornando inútil, porque ele colou isso ali apenas para ele ver e acabaram aprovando sem pedir para ele remover.
Visual evidences :framed_picture: https://github.com/devhatt/octopost/assets/30197026/b946a27a-3f72-48c4-bef7-a9771104f849 https://github.com/devhatt/octopost/assets/30197026/c160f236-d758-4b11-9ff9-32f4f47acbfe
github-actions[bot] commented 2 months ago

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 41.43% 150 / 362
🔵 Statements 43% 169 / 393
🔵 Functions 50.65% 77 / 152
🔵 Branches 61.65% 82 / 133
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
Changed Files
src/pages/home/components/Sidebar/Sidebar.tsx 100% 100% 100% 100%
Generated in workflow #1143
alvarogfn commented 2 months ago

image

bota 100% de coverage pra nois

JpBurgarelli commented 2 months ago

Basicamente, nesta Task, apenas removi o searchClue, suas importações e lógicas derivadas do sideBar onde era importado e fiz a covarage chegar a 100%.

Porém, acabei tendo que remover também uma função/botão implementado pelo Zoldyck no sideBar, pois o ele estava ali, única e exclusivamente, para que, quando esse botão fosse clicado, as estilizações que ele fez para o sideBar na versão mobile fossem aplicadas/chamadas. Então, acabava se tornando inútil, porque ele colou isso ali apenas para ele ver e acabaram aprovando sem pedir para ele remover.