Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sample boilerplate framework agnostic #21

Closed
wants to merge 1 commit into from
Closed

sample boilerplate framework agnostic #21

wants to merge 1 commit into from

Conversation

dhulke
Copy link

@dhulke dhulke commented May 5, 2020

Descrição das Mudanças

Sugestão de boilerplate. Essa estrutura torna o código independente de framework. Concentra as regras de negócio nas entidades e "use cases". Essas por sua vez recebem os repositorios como parametro via DI. Os use cases são utilizados pelos entry points (frameworks web, command line e outros). Com essa estrutura, não misturamos a regra de negócio com nenhuma dependencia externa como frameworks e banco.

mocha test/unit/usecase/deliveryperson/ListDeliveryPersonByIdUseCase.js
mocha test/web/deliverypersonendpoints.js

@rodrigondec rodrigondec added Status: Fora do Plano Essa sugestão não está dentro dos nossos planos, mas pode estar no futuro ;). Status: NÃO MERGEAR! Mesmo que tudo esteja verde, não faça o merge deste PR. Status: Não verificado Quando uma nova issue é aberta e ainda não foi analisada labels May 5, 2020
@rodrigondec
Copy link
Member

rodrigondec commented May 5, 2020

@dhulke obrigado pela sua contribuição.

Porém seu PR está fora dos nossos padrões aceitáveis.

que ainda estão em discussão mas que já temos algumas definições #2

Além disso sua contribuição não está condizente com as discussões/decisões (tanto nos issues quanto na reunião feita dia 03/05/2020) que tivemos sobre o boilerplate.

Devido ao fato de que ela já possuí implementação lógica de um model DeliveryPerson mas não chegamos nem na parte de modelagem dos dados.

@dhulke
Copy link
Author

dhulke commented May 5, 2020

No #17 os requisitos do #2 não pareceram um impedimento além é claro de não seguir os pontos da reunião. Você se importa de apontar o ponto específico de recusa aqui?

E sobre o model, eu fiz como forma de mostrar onde cada peça do código se encaixa. Não entendi sua reclamação.

@rodrigondec
Copy link
Member

Mediante discussão feita no discord esse PR será fechado e as considerações dele serão baseadas/feitas no #17.

@rodrigondec rodrigondec closed this May 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Fora do Plano Essa sugestão não está dentro dos nossos planos, mas pode estar no futuro ;). Status: NÃO MERGEAR! Mesmo que tudo esteja verde, não faça o merge deste PR. Status: Não verificado Quando uma nova issue é aberta e ainda não foi analisada
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants