Merge pull request #2 from dxe/jake/add-tally-endpoint

fix url param
dxe · Jun 21, 2024 · 5611bff · 5611bff
2 parents 07a956e + 6ae072f
commit 5611bff
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/service/handlers.go b/service/handlers.go
@@ -9,7 +9,6 @@ import (
 
 	"github.com/dxe/helptheducks.com/service/config"
 	"github.com/dxe/helptheducks.com/service/model"
-	"github.com/go-chi/chi/v5"
 )
 
 type CreateMessageInput struct {
@@ -100,7 +99,12 @@ type GetTallyResp struct {
 }
 
 func getTallyHandler(w http.ResponseWriter, r *http.Request) {
-	campaign := chi.URLParam(r, "campaign")
+	queryParams := r.URL.Query()
+	campaign := queryParams.Get("campaign")
+	if (campaign == "") {
+		http.Error(w, "missing campaign", http.StatusBadRequest)
+		return
+	}
 
 	tally, err := model.GetTally(db, campaign)
 	if err != nil {