Refatoração do controller de carrinhos #378
Conversation
…ethod from carrinhosService
|
Acredito que tive o mesmo problema de não encontrar a branch da última vez. Existe uma forma de não ter esse problema no futuro? |
|
@edumaxsantos eu vou verificar isso e tentar ajustar |
|
@edumaxsantos consegue me dar acesso de colaborador para o seu repositório? Pois aí faço um commit no seu fork corrigindo o checkout e já serve para validar se funcionou. |
|
@PauloGoncalvesBH feito. Adicionei como colaborador. |
PauloGoncalvesBH
force-pushed
the
refactoring/carrinhos-controller
branch
2 times, most recently
from
e8cc124
to
f5d3bb5
Compare
|
@edumaxsantos posso fazer o review ou ainda está em draft? |
|
@PauloGoncalvesBH pode fazer review. |
|
O clone foi ajustado, porém de nada adiantou pois os 2 jobs estão a falhar por não ter acesso a 2 secrets.
|
|
Estou olhando como fizemos da última vez, mas não me recordo bem, só lembro que ocorreu algum problema parecido. |
|
@edumaxsantos acabei de revisar e a sua alteração ficou ótima, simplificou a controller de carrinhos, que estava bem confusa, facilitando uma manutenção futura e na criação de testes unitários do services, que pretendo criar em breve. |
This reverts commit c678f63.
|
🎉 This PR is included in version 2.26.5 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Description
Continuando as refatorações iniciadas em #303, realizei uma refatoração no arquivo inteiro de carrinhosController.
How can the user experience this change?
N/A
Documentation
N/A
Related Issues
Related to #208
PR Tasks