diff --git a/app/http/endpoints/api/panel/panelcreate.go b/app/http/endpoints/api/panel/panelcreate.go index 82bc156..60e79cd 100644 --- a/app/http/endpoints/api/panel/panelcreate.go +++ b/app/http/endpoints/api/panel/panelcreate.go @@ -174,7 +174,7 @@ func CreatePanel(ctx *gin.Context) { ctx.JSON(200, gin.H{ "success": true, - "panel_id": strconv.Itoa(panelId), + "panel_id": panelId, }) } diff --git a/app/http/endpoints/api/panel/paneldelete.go b/app/http/endpoints/api/panel/paneldelete.go index 21f55ec..c0b1fb1 100644 --- a/app/http/endpoints/api/panel/paneldelete.go +++ b/app/http/endpoints/api/panel/paneldelete.go @@ -20,7 +20,7 @@ func DeletePanel(ctx *gin.Context) { return } - panelId, err := strconv.Atoi(ctx.Param("id")) + panelId, err := strconv.Atoi(ctx.Param("panelid")) if err != nil { ctx.AbortWithStatusJSON(400, gin.H{ "success": false, diff --git a/app/http/endpoints/api/panel/panelupdate.go b/app/http/endpoints/api/panel/panelupdate.go index b24a791..c8c7ae5 100644 --- a/app/http/endpoints/api/panel/panelupdate.go +++ b/app/http/endpoints/api/panel/panelupdate.go @@ -33,7 +33,7 @@ func UpdatePanel(ctx *gin.Context) { return } - panelId, err := strconv.Atoi(ctx.Param("id")) + panelId, err := strconv.Atoi(ctx.Param("panelid")) if err != nil { ctx.AbortWithStatusJSON(400, utils.ErrorJson(err)) return @@ -122,14 +122,8 @@ func UpdatePanel(ctx *gin.Context) { newMessageId := existing.MessageId if shouldUpdateMessage { - // delete old message - if err := rest.DeleteMessage(botContext.Token, botContext.RateLimiter, existing.ChannelId, existing.MessageId); err != nil { - ctx.AbortWithStatusJSON(500, gin.H{ - "success": false, - "error": err.Error(), - }) - return - } + // delete old message, ignoring error + _ = rest.DeleteMessage(botContext.Token, botContext.RateLimiter, existing.ChannelId, existing.MessageId) premiumTier := rpc.PremiumClient.GetTierByGuildId(guildId, true, botContext.Token, botContext.RateLimiter) newMessageId, err = data.sendEmbed(&botContext, existing.Title, existing.CustomId, existing.ReactionEmote, premiumTier > premium.None) @@ -147,22 +141,6 @@ func UpdatePanel(ctx *gin.Context) { return } - - // Add reaction - if err = rest.CreateReaction(botContext.Token, botContext.RateLimiter, data.ChannelId, newMessageId, emoji); err != nil { - var unwrapped request.RestError - if errors.As(err, &unwrapped) && unwrapped.StatusCode == 403 { - ctx.AbortWithStatusJSON(500, gin.H{ - "success": false, - "error": "I do not have permission to add reactions in the specified channel", - }) - } else { - // TODO: Most appropriate error? - ctx.AbortWithStatusJSON(500, utils.ErrorJson(err)) - } - - return - } } // Store in DB @@ -178,6 +156,7 @@ func UpdatePanel(ctx *gin.Context) { ReactionEmote: emoji, WelcomeMessage: data.WelcomeMessage, WithDefaultTeam: utils.ContainsString(data.Teams, "default"), + CustomId: existing.CustomId, } if err = dbclient.Client.Panel.Update(panel); err != nil { @@ -237,6 +216,6 @@ func UpdatePanel(ctx *gin.Context) { } ctx.JSON(200, gin.H{ - "success": true, + "success": true, }) } diff --git a/app/http/server.go b/app/http/server.go index ffbd40d..d76a788 100644 --- a/app/http/server.go +++ b/app/http/server.go @@ -101,8 +101,8 @@ func StartServer() { guildAuthApiAdmin.GET("/panels", api_panels.ListPanels) guildAuthApiAdmin.PUT("/panels", api_panels.CreatePanel) - guildAuthApiAdmin.PUT("/panels/:id", api_panels.UpdatePanel) - guildAuthApiAdmin.DELETE("/panels/:id", api_panels.DeletePanel) + guildAuthApiAdmin.PUT("/panels/:panelid", api_panels.UpdatePanel) + guildAuthApiAdmin.DELETE("/panels/:panelid", api_panels.DeletePanel) guildAuthApiAdmin.GET("/multipanels", api_panels.MultiPanelList) guildAuthApiAdmin.POST("/multipanels", api_panels.MultiPanelCreate) diff --git a/public/templates/includes/paneleditmodal.tmpl b/public/templates/includes/paneleditmodal.tmpl index 0179355..a7b893c 100644 --- a/public/templates/includes/paneleditmodal.tmpl +++ b/public/templates/includes/paneleditmodal.tmpl @@ -98,7 +98,7 @@ - + @@ -114,14 +114,14 @@