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

doc(Documentar API) 📝 Remover o serviço e o contrato do pedido, melhorar a lógica de negócios #11

Merged
merged 3 commits into from
Dec 26, 2023

Conversation

diegosneves
Copy link
Owner

@diegosneves diegosneves commented Dec 26, 2023

Solicitação de Pull Request

Status

  • In Progress
  • Ready to Merging

Tipo

  • Release
  • Feature
  • Technical Debt
  • Fix
  • Test
  • Refactor
  • Documentation
  • Performance

Descrição:

  • Remoção do Serviço e Contrato de Pedido:
    O serviço de pedidos e seu respetivo contrato, juntamente com suas exceções, foram removidos.

  • Melhoria na Lógica de Negócio:
    A lógica de negócio foi aperfeiçoada nas classes NoDiscountStrategy, NotificationEmailMapper e SplitInvoiceServiceTest.

  • Inclusão de Novos Casos de Teste:
    Novos casos de teste foram adicionados para validar essas mudanças.


  • Limpeza da Documentação do Código:
    A documentação do código foi aprimorada, incluindo a remoção de alguns asteriscos e linhas desnecessários.

  • Correção da Tag do Autor:
    A tag do autor foi corrigida em algumas classes.

  • Inclusão de Hiperlink:
    Foi inserido um hiperlink no arquivo OpenApiConfig.

  • Otimização de Importações:
    No NotificationEmailMapper e SplitInvoiceResponse, algumas importações que não estavam sendo utilizadas foram removidas, contribuindo para a otimização do código.

…rar a lógica de negócios

- **Remoção do Serviço e Contrato de Pedido**:
O serviço de pedidos e seu respetivo contrato, juntamente com suas exceções, foram removidos.

- **Melhoria na Lógica de Negócio**:
A lógica de negócio foi aperfeiçoada nas classes `NoDiscountStrategy`, `NotificationEmailMapper` e `SplitInvoiceServiceTest`.

- **Inclusão de Novos Casos de Teste**:
Novos casos de teste foram adicionados para validar essas mudanças.
…o do fluxo de trabalho

- **Remoção do Código Comentado para a Construção do Docker no Workflow**:
O código de ação para a construção e envio do Docker que estava comentado, foi removido do arquivo de workflow das ações do GitHub.
… importações não utilizadas

- **Limpeza da Documentação do Código**:
A documentação do código foi aprimorada, incluindo a remoção de alguns asteriscos e linhas desnecessários.

- **Correção da Tag do Autor**:
A tag do autor foi corrigida em algumas classes.

- **Inclusão de Hiperlink**:
Foi inserido um hiperlink no arquivo OpenApiConfig.

- **Otimização de Importações**:
No `NotificationEmailMapper` e `SplitInvoiceResponse`, algumas importações que não estavam sendo utilizadas foram removidas, contribuindo para a otimização do código.
@diegosneves diegosneves merged commit 733e9c3 into develop Dec 26, 2023
1 check passed
@diegosneves diegosneves deleted the feat/documentation branch December 26, 2023 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant