DevdouglasGfs / taqtile

A repository created with the purpose of make the project proposed in the Technical Test stage.
https://taqtile.vercel.app
0 stars 0 forks source link

Feat: Backend integration #14

Closed DevdouglasGfs closed 2 months ago

DevdouglasGfs commented 2 months ago

Back-end integration

Integrate the front-end with the back-end(via API) to validate user login.

it solves step 3/5 of issue #11

DevdouglasGfs commented 2 months ago

Tela de login correspondente

Screenshot_20240626-123320~2 Screenshot_20240626-123353~2

DevdouglasGfs commented 2 months ago

As coisas que eu já resolvi em outras PRs eu devo resolver nessa aqui ou trazer @racsandoval ?

vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
taqtile ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 2, 2024 11:33pm
SamuelSsc commented 2 months ago

As coisas que eu já resolvi em outras PRs eu devo resolver nessa aqui ou trazer @racsandoval ?

Opa @DevdouglasGfs Nesse caso aqui o que foi comentado deve ser ajustado no pr em que o comentário surgiu e com o rebase o problema estará resolvido nos pr's posteriores, isso pq a pessoa que revisou o código não vai lembrar de ver a correção do problema em vários pr's e acompanhar se foi realmente arrumado algo, por isso é sempre arrumado no pr em que o comentário surgiu.

DevdouglasGfs commented 2 months ago

As coisas que eu já resolvi em outras PRs eu devo resolver nessa aqui ou trazer @racsandoval ?

Opa @DevdouglasGfs Nesse caso aqui o que foi comentado deve ser ajustado no pr em que o comentário surgiu e com o rebase o problema estará resolvido nos pr's posteriores, isso pq a pessoa que revisou o código não vai lembrar de ver a correção do problema em vários pr's e acompanhar se foi realmente arrumado algo, por isso é sempre arrumado no pr em que o comentário surgiu.

Muito obrigado pela explicação! Vou fazer as alterações de acordo

DevdouglasGfs commented 2 months ago

apliquei cada mudança solicitada @SamuelSsc @racsandoval @walteraandrade