devhatt / pet-dex-backend

MIT License
45 stars 59 forks source link

issue: documentação da rota pet PATCH (#175) #192

Closed EdiltonOliveira closed 2 months ago

EdiltonOliveira commented 2 months ago

Closes #175

Feature - feat: documentação da rota PATCH pet - fix: recuperando rota de atualização de pet excluído no merge: [**[Listar pets do usuário #67](https://github.com/devhatt/pet-dex-backend/pull/67/files)**] - fix: correção de inconsistência de propriedade declarada no PetUpdateDto com banco de dados - fix: alteração das tags json para consistência no envio e recebimento de dados pelo DB.
Bugfix - **Description** TÓPICO: recuperando rota de atualização de pet excluído no merge Durante o processo de documentação do endpoint … identificou-se a ausência da rota responsável pelo PATCH contendo {userID} e {petID}. Uma verificação nos commits, foi identificado que a rota que fornecia esses dados já tinha sido criada, mas a posteriori ocorreu sua exclusão. Isso passou despercebido no merge: [**[Listar pets do usuário #67](https://github.com/devhatt/pet-dex-backend/pull/67/files)**] ![davidspacer_routes_update_pet](https://github.com/user-attachments/assets/7a10e4f4-b2e1-4690-8240-91ec2fc4a912) fix: inconsistência de propriedade declarada no PetUpdateDto com banco de dados {"description":"failed to update pet with ID f6254629-6033-11ef-9d30-0242ac130003: error updating pet: Error 1054 (42S22): Unknown column 'needed' in 'field list' \\n"} Solução: Adicionei a referência correta a propriedade ao arquivo `pet_repository.go` . Nesse caso, o script sql ficou `p.neededSpecialCare` e a referência ao scan `&pet.NeedSpecialCare.Needed` . Realizei a reescrita das tags de `needed` / `description`para `neededSpecialCare` e `descriptionSpecialCare` . - **Cause** N/A - **Solution** N/A
Changelog N/A
Visual evidences :framed_picture: a. ***Atualização do pet no banco de dados via swagger. Retorno statuscode 200.*** ![patch_pet_swagger_screenshot](https://github.com/user-attachments/assets/b23605a9-68e3-43e7-ae4e-6c0a729e24ed) b. ***Dados pets atualizados no DB*** ![patch_pet_database](https://github.com/user-attachments/assets/498434bb-f4d4-402f-90d5-3760b7163301)
Checklist - [x] Issue linked - [ ] Build working correctly - [ ] Tests created
Additional info N/A
EdiltonOliveira commented 2 months ago

Caro, revisor.

A correção para o problema de CI em destaque nesse pull request, só está disponível através do pull #189 . Essa solicitação atualmente está pendente de aprovação.