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

Criar Delete de ONG - Issue 113 #161

Merged
merged 8 commits into from
Aug 9, 2024
Merged

Conversation

matheus-webp
Copy link
Contributor

Closes #113

Feature

Um endpoint DELETE que deleta as ongs no banco, usando o conceito de soft delete enquanto estou autenticado.

Checklist
  • Issue linked
  • Build working correctly
  • Tests created

Copy link
Member

@zoldyzdk zoldyzdk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tem que dar uma revisada nos teus testes, eles não passaram no CI

Copy link
Member

@zoldyzdk zoldyzdk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eu não vi a rota pra deletar a ong.

@PiluVitu PiluVitu self-requested a review August 6, 2024 13:43
@devlefel devlefel merged commit b21d0ed into devhatt:main Aug 9, 2024
4 checks passed
EdiltonOliveira pushed a commit to EdiltonOliveira/pet-dex-backend that referenced this pull request Aug 10, 2024
* feat: add deletedAt field legal_persons

* feat: add delete ong repository

* feat: add delete ong usecase

* feat: add delete dto

* feat: add delete ong controller

* feat: add delete ong usecase test

* fix: fix tests for OngUsecase's Delete method

* feat: add delete ong route
devlefel pushed a commit that referenced this pull request Aug 14, 2024
* feat: swagger implementation for API documentation (issue #119)

* feat: add swag command to Makefile and FindbyID endpoint documentation pilot example

* refact: swagger implementation for API documentation (issue #119)

* Implementação de documentação completa para rota Pet (createPet)

* Criação de docs para orientação de acesso e geração de novas documentações.

* Docs: Correção documentação swago_documentation.md #119

* Issue 129 (#164)

* feat: add Docker image push to Docker Hub step in CI pipeline

* feat: add Docker Push workflow and version images using commit SHA

* update docker push

* feat: update vars and pull request branch

* feat: update condition

* feat: update docker action version

* chore: ajust action trigger

* chore: ajust action build

* chore: ajust compose name

* chore: ajust workflow

---------

Co-authored-by: David Ambrozio <davidambroziodev@gmail.com>

* refact: altera a documentação swagger para uma rota pública e outras correções.

* Criar Delete de ONG - Issue 113 (#161)

* feat: add deletedAt field legal_persons

* feat: add delete ong repository

* feat: add delete ong usecase

* feat: add delete dto

* feat: add delete ong controller

* feat: add delete ong usecase test

* fix: fix tests for OngUsecase's Delete method

* feat: add delete ong route

* issue-119: implementação das alterações solicitadas

* Correção de bug criado pela rota delete{ongID}

* alteração de diretório da documentação swag.

* implementação de tags example para documentação via swag

* modificação do arquivo makefile para geração de documentação no diretório swag.

---------

Co-authored-by: Paulo Victor <pilutechinformatica@gmail.com>
Co-authored-by: David Ambrozio <davidambroziodev@gmail.com>
Co-authored-by: Matheus Melo <117240363+matheus-webp@users.noreply.github.com>
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.

Criar Delete de ONG
3 participants