From 5915e3ca7b44148de1d0730c63ceed57c140ff81 Mon Sep 17 00:00:00 2001 From: Victor Lacasse-Beaudoin Date: Thu, 8 Jun 2023 20:47:39 -0400 Subject: [PATCH] Retirer route GET /v2/membres/:membre_id MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Cleanup code désuet (et redondant avec agecem/bottin) en lien avec le concept de Membre --- cmd/api.go | 2 -- handlers/membre.go | 56 ------------------------------------------ responses/responses.go | 10 -------- 3 files changed, 68 deletions(-) delete mode 100644 handlers/membre.go diff --git a/cmd/api.go b/cmd/api.go index 018fbc1..03b6e30 100644 --- a/cmd/api.go +++ b/cmd/api.go @@ -59,8 +59,6 @@ var apiCmd = &cobra.Command{ e.GET("/v2/health/", handlers.GetHealth) - e.GET("/v2/membres/:membre_id/", handlers.GetMembre) - e.GET("/v2/transactions/", handlers.GetTransactions) e.POST("/v2/transactions/", handlers.PostTransactions) diff --git a/handlers/membre.go b/handlers/membre.go deleted file mode 100644 index 67b5215..0000000 --- a/handlers/membre.go +++ /dev/null @@ -1,56 +0,0 @@ -package handlers - -import ( - "net/http" - - "github.com/labstack/echo/v4" - "github.com/spf13/viper" - - "git.agecem.com/agecem/bottin-agenda/responses" - bottindata "git.agecem.com/agecem/bottin/v5/data" -) - -func GetMembre(c echo.Context) error { - bottinApiKey := viper.GetString("bottin.api.key") - bottinApiHost := viper.GetString("bottin.api.host") - bottinApiProtocol := viper.GetString("bottin.api.protocol") - bottinApiPort := viper.GetInt("bottin.api.port") - - // Using bottin's API client - bottinConnection := bottindata.NewApiClient( - bottinApiKey, - bottinApiHost, - bottinApiProtocol, - bottinApiPort, - ) - - membreID := c.Param("membre_id") - - getMembreResponse := responses.GetMembreResponse{} - - membre, err := bottinConnection.GetMembre(membreID) - if err != nil { - getMembreResponse.Message = err.Error() - - var statusCode int - - switch err.Error() { - case "Veuillez fournir un numéro étudiant à rechercher": - statusCode = http.StatusBadRequest - case "Ce numéro étudiant ne correspond à aucunE membre": - statusCode = http.StatusNotFound - default: - statusCode = http.StatusInternalServerError - } - - getMembreResponse.Data.Membre = membre - - return c.JSON(statusCode, getMembreResponse) - } - - getMembreResponse.Data.Membre = membre - - getMembreResponse.Message = "Read successful" - - return c.JSON(http.StatusOK, getMembreResponse) -} diff --git a/responses/responses.go b/responses/responses.go index fcadcce..7ac2507 100644 --- a/responses/responses.go +++ b/responses/responses.go @@ -2,22 +2,12 @@ package responses import ( "git.agecem.com/agecem/bottin-agenda/models" - bottinmodels "git.agecem.com/agecem/bottin/v5/models" ) type PostSeedResponse struct { Message string `json:"message"` } -type GetMembreResponseData struct { - Membre bottinmodels.Membre `json:"membre"` -} - -type GetMembreResponse struct { - Message string `json:"message"` - Data GetMembreResponseData `json:"data"` -} - type PostTransactionsResponseData struct { Transactions []models.Transaction `json:"transactions"` }