Skip to content

Refatoração do controller de carrinhos#378

Merged
PauloGoncalvesBH merged 9 commits intoServeRest:trunkfrom
edumaxsantos:refactoring/carrinhos-controller
Oct 18, 2022
Merged

Refatoração do controller de carrinhos#378
PauloGoncalvesBH merged 9 commits intoServeRest:trunkfrom
edumaxsantos:refactoring/carrinhos-controller

Conversation

@edumaxsantos
Copy link
Copy Markdown
Collaborator

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

  • Has been related to an issue?
  • Have tests been added/updated?
  • Has Aglio documentation been added/updated?

@PauloGoncalvesBH PauloGoncalvesBH self-requested a review October 18, 2022 21:44
@edumaxsantos
Copy link
Copy Markdown
Collaborator Author

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?

@PauloGoncalvesBH
Copy link
Copy Markdown
Member

@edumaxsantos eu vou verificar isso e tentar ajustar

@PauloGoncalvesBH
Copy link
Copy Markdown
Member

@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.

@edumaxsantos
Copy link
Copy Markdown
Collaborator Author

@PauloGoncalvesBH feito. Adicionei como colaborador.

@PauloGoncalvesBH PauloGoncalvesBH marked this pull request as ready for review October 18, 2022 22:55
@PauloGoncalvesBH PauloGoncalvesBH marked this pull request as draft October 18, 2022 22:56
@PauloGoncalvesBH PauloGoncalvesBH force-pushed the refactoring/carrinhos-controller branch 2 times, most recently from e8cc124 to f5d3bb5 Compare October 18, 2022 22:57
@PauloGoncalvesBH
Copy link
Copy Markdown
Member

@edumaxsantos posso fazer o review ou ainda está em draft?

@edumaxsantos
Copy link
Copy Markdown
Collaborator Author

@PauloGoncalvesBH pode fazer review.

@PauloGoncalvesBH
Copy link
Copy Markdown
Member

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.
De acordo com a documentação do github actions os secrets não são passados para forks, o que faz sentido em termos de segurança.
https://docs.github.com/pt/actions/security-guides/encrypted-secrets#using-encrypted-secrets-in-a-workflow

image

@edumaxsantos
Copy link
Copy Markdown
Collaborator Author

Estou olhando como fizemos da última vez, mas não me recordo bem, só lembro que ocorreu algum problema parecido.

@PauloGoncalvesBH
Copy link
Copy Markdown
Member

@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.
Muito obrigado mano <3

@PauloGoncalvesBH PauloGoncalvesBH marked this pull request as ready for review October 18, 2022 23:13
@PauloGoncalvesBH PauloGoncalvesBH merged commit 325cdf7 into ServeRest:trunk Oct 18, 2022
@PauloGoncalvesBH PauloGoncalvesBH deleted the refactoring/carrinhos-controller branch October 18, 2022 23:22
@PauloGoncalvesBH
Copy link
Copy Markdown
Member

🎉 This PR is included in version 2.26.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants