Ocultar Sessão - Carossel Home

SouJunior / vagas-webapp

Frontend do Projeto de Vagas do @SouJunior

https://vagas.soujunior.tech

GNU General Public License v3.0

222 stars 35 forks source link

Ocultar Sessão - Carossel Home #302

Closed brunowzz closed 1 month ago

brunowzz commented 2 months ago

Ocultar Sessão - Carrossel - #486

Descrição

Ocultei o carrossel de profissões que havia na home.

Mudanças

[x] Ocultei o carrosel das profissões
[x] Removi a contagem de vagas

Prints

netlify[bot] commented 2 months ago

Deploy Preview for vagas-front-end ready!

Name	Link
Latest commit	b6c6e4d3442bcc436b88d00376a3bd8c99bd00f5
Latest deploy log	https://app.netlify.com/sites/vagas-front-end/deploys/664be4c7ada3d700082bca81
Deploy Preview	https://deploy-preview-302--vagas-front-end.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] commented 2 months ago

Deploy Preview for vagas-soujunior failed. Why did it fail? →

Name	Link
Latest commit	5b06e1417db986b25c5fdc3fabec9acb5032886f
Latest deploy log	https://app.netlify.com/sites/vagas-soujunior/deploys/663d664a5441030008f1de0d

sonarcloud[bot] commented 1 month ago

Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

RafaelSilva-si commented 1 month ago

Fala @brunowzz Brunão, valeu pelo PR!

Eu dei uma olhada na US, e percebi que eles pediram para somente ocultar, e não remover o componente de carrossel, pelo que entendi, iremos utilizar futuramente.

Acredito que o mais correto, seria deixar todos os componentes, porém, só remover o componente do arquivo de Home, assim o componente não iria aparecer, mas teremos ele pronto para uso, quando precisar.

O que acha?

brunowzz commented 1 month ago

Opa @RafaelSilva-si, eu fiz assim pois foi uma prática que meu antigo mentor me falou pra fazer. Mas creio que não tem problema, pois é só voltar no histórico de commits e pegar os arquivos novamente! Fora que o Carrossel não estava com um comportamento desejado, o nosso antigo QA até pensou em reportar ele com alguns bugs.

RafaelSilva-si commented 1 month ago

@brunowzz

Na minha visão, seria mais prático só remover a chamada do componente da Home, e quando precisarem só chamar novamente, do que procurar no histórico e etc... Mas se achar que seja mais fácil, pode dar sequência assim. 💯

Um outro dúvida, é que esse PR se trata de ocultar o Carrossel, mas existem algumas alterações como o arquivo src/Mocks/CurriculoItems.tsx, gostaria de saber se ele realmente deveria estar nesse PR.

Abraços :)