From 173d56d399002e5cdc2a9a14cbca76482f5195e4 Mon Sep 17 00:00:00 2001 From: Robin Dittmar Date: Tue, 1 Sep 2026 15:44:27 +0200 Subject: [PATCH] fix: /lists/items now consistent in singular/plural usage --- internal/api/handler/list.go | 6 +++--- internal/api/router/router.go | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/internal/api/handler/list.go b/internal/api/handler/list.go index 9889809..ad2863f 100644 --- a/internal/api/handler/list.go +++ b/internal/api/handler/list.go @@ -243,7 +243,7 @@ func (h *ListHandler) RemoveUserFromList(w http.ResponseWriter, r *http.Request) // @Success 204 {object} nil // @Error 400 {object} response.ErrorResponse "failed to decode request body" // @Error 500 {object} response.ErrorResponse "failed to create list item" -// @Router /lists/item [post] +// @Router /lists/items [post] func (h *ListHandler) CreateListItem(w http.ResponseWriter, r *http.Request) { ctx := r.Context() @@ -283,7 +283,7 @@ func (h *ListHandler) CreateListItem(w http.ResponseWriter, r *http.Request) { // @Success 204 // @Error 400 {object} response.ErrorResponse "failed to decode request url" // @Error 500 {object} response.ErrorResponse "failed to delete list item" -// @Router /lists/item/{id} [delete] +// @Router /lists/items/{id} [delete] func (h *ListHandler) DeleteListItem(w http.ResponseWriter, r *http.Request) { ctx := r.Context() @@ -324,7 +324,7 @@ func (h *ListHandler) DeleteListItem(w http.ResponseWriter, r *http.Request) { // @Error 400 {object} response.ErrorResponse "failed to decode request body" // @Error 401 {object} response.ErrorResponse "not authorized" // @Error 500 {object} response.ErrorResponse "failed to update list item" -// @Router /lists/item [put] +// @Router /lists/items [put] func (h *ListHandler) UpdateListItem(w http.ResponseWriter, r *http.Request) { ctx := r.Context() diff --git a/internal/api/router/router.go b/internal/api/router/router.go index 0296f3e..16f8ac8 100644 --- a/internal/api/router/router.go +++ b/internal/api/router/router.go @@ -57,9 +57,9 @@ func NewMux(cfg Config) http.Handler { apiMux.Handle("GET /lists", protected(listHandler.GetLists)) apiMux.Handle("POST /lists/user", protected(listHandler.AddUserToList)) apiMux.Handle("DELETE /lists/user", protected(listHandler.RemoveUserFromList)) - apiMux.Handle("POST /lists/item", protected(listHandler.CreateListItem)) - apiMux.Handle("DELETE /lists/item/{id}", protected(listHandler.DeleteListItem)) - apiMux.Handle("PUT /lists/item", protected(listHandler.UpdateListItem)) + apiMux.Handle("POST /lists/items", protected(listHandler.CreateListItem)) + apiMux.Handle("DELETE /lists/items/{id}", protected(listHandler.DeleteListItem)) + apiMux.Handle("PUT /lists/items", protected(listHandler.UpdateListItem)) apiMux.Handle("POST /lists/items/{id}", protected(listHandler.SetListItemCompleted)) apiMux.Handle("GET /lists/{id}", protected(listHandler.GetListItems))