Working on adding complete endpoint

backend/api_test.go

@@ -52,9 +52,7 @@ func TestGetUserBadId(t *testing.T) {
 func TestGetUserAllowanceWhenNoAllowancePresent(t *testing.T) {
 	e := startServer(t)
 	result := e.GET("/user/1/allowance").Expect().Status(200).JSON().Array()
-	result.Length().IsEqual(1)
+	result.Length().IsEqual(0)
-	item := result.Value(0).Object()
-	item.Value("id").IsEqual(0)
 }
 
 func TestGetUserAllowance(t *testing.T) {
@@ -70,8 +68,8 @@ func TestGetUserAllowance(t *testing.T) {
 
 	// Validate allowance
 	result := e.GET("/user/1/allowance").Expect().Status(200).JSON().Array()
-	result.Length().IsEqual(2)
+	result.Length().IsEqual(1)
-	item := result.Value(1).Object()
+	item := result.Value(0).Object()
 	item.Value("id").IsEqual(1)
 	item.Value("name").IsEqual(TestAllowanceName)
 	item.Value("target").IsEqual(5000)
@@ -116,9 +114,9 @@ func TestCreateUserAllowance(t *testing.T) {
 		Status(200).
 		JSON().Array()
 
-	allowances.Length().IsEqual(2)
+	allowances.Length().IsEqual(1)
 
-	allowance := allowances.Value(1).Object()
+	allowance := allowances.Value(0).Object()
 	allowance.Value("id").IsEqual(allowanceId)
 	allowance.Value("name").IsEqual(TestAllowanceName)
 	allowance.Value("target").IsEqual(5000)
@@ -210,12 +208,7 @@ func TestDeleteUserAllowance(t *testing.T) {
 		Expect().
 		Status(200).
 		JSON().Array()
-	allowances.Length().IsEqual(1)
+	allowances.Length().IsEqual(0)
-}
-
-func TestDeleteUserRestAllowance(t *testing.T) {
-	e := startServer(t)
-	e.DELETE("/user/1/allowance/0").Expect().Status(400)
 }
 
 func TestDeleteUserAllowanceNotFound(t *testing.T) {
@@ -361,10 +354,10 @@ func TestGetTaskWhenNoTasks(t *testing.T) {
 	result.Length().IsEqual(0)
 }
 
-func createTestTaskWithAmount(e *httpexpect.Expect, amount int) int {
+func createTestTask(e *httpexpect.Expect) int {
 	requestBody := map[string]interface{}{
 		"name":   "Test Task",
-		"reward": amount,
+		"reward": 100,
 	}
 	return int(e.POST("/tasks").WithJSON(requestBody).Expect().Status(201).JSON().Object().Value("id").Number().Raw())
 }
@@ -538,14 +531,10 @@ func TestPutAllowanceById(t *testing.T) {
 
 func TestCompleteTask(t *testing.T) {
 	e := startServer(t)
-	taskId := createTestTaskWithAmount(e, 101)
+	taskId := createTestTask(e)
 
 	e.GET("/tasks").Expect().Status(200).JSON().Array().Length().IsEqual(1)
 
-	// Update rest allowance
-	e.PUT("/user/1/allowance/0").WithJSON(UpdateAllowanceRequest{
-		Weight: 25,
-	}).Expect().Status(200)
 	// Create two allowance goals
 	e.POST("/user/1/allowance").WithJSON(CreateAllowanceRequest{
 		Name:   "Test Allowance 1",
@@ -557,137 +546,24 @@ func TestCompleteTask(t *testing.T) {
 		Target: 1000,
 		Weight: 25,
 	}).Expect().Status(201)
-
-	// Complete the task
-	e.POST("/task/" + strconv.Itoa(taskId) + "/complete").Expect().Status(200)
-
-	// Verify the task is marked as completed
-	e.GET("/task/" + strconv.Itoa(taskId)).Expect().Status(404)
-
-	// Verify the allowances are updated for user 1
-	allowances := e.GET("/user/1/allowance").Expect().Status(200).JSON().Array()
-	allowances.Length().IsEqual(3)
-	allowances.Value(0).Object().Value("id").Number().IsEqual(0)
-	allowances.Value(0).Object().Value("progress").Number().IsEqual(26)
-	allowances.Value(1).Object().Value("id").Number().IsEqual(1)
-	allowances.Value(1).Object().Value("progress").Number().IsEqual(50)
-	allowances.Value(2).Object().Value("id").Number().IsEqual(2)
-	allowances.Value(2).Object().Value("progress").Number().IsEqual(25)
-
-	// And also for user 2
-	allowances = e.GET("/user/2/allowance").Expect().Status(200).JSON().Array()
-	allowances.Length().IsEqual(1)
-	allowances.Value(0).Object().Value("id").Number().IsEqual(0)
-	allowances.Value(0).Object().Value("progress").Number().IsEqual(101)
-
-	for userId := 1; userId <= 2; userId++ {
-		userIdStr := strconv.Itoa(userId)
-		// Ensure the history got updated
-		history := e.GET("/user/" + userIdStr + "/history").Expect().Status(200).JSON().Array()
-		history.Length().IsEqual(1)
-		history.Value(0).Object().Value("allowance").Number().IsEqual(101)
-		history.Value(0).Object().Value("timestamp").String().AsDateTime().InRange(getDelta(time.Now(), 2.0))
-	}
-}
-
-func TestCompleteTaskAllowanceWeightsSumTo0(t *testing.T) {
-	e := startServer(t)
-	taskId := createTestTaskWithAmount(e, 101)
-
-	e.GET("/tasks").Expect().Status(200).JSON().Array().Length().IsEqual(1)
-
-	// Update rest allowance
 	e.PUT("/user/1/allowance/0").WithJSON(UpdateAllowanceRequest{
-		Weight: 0,
+		Weight: 25,
 	}).Expect().Status(200)
-	// Create an allowance goal
-	createTestAllowance(e, "Test Allowance 1", 1000, 0)
 
 	// Complete the task
 	e.POST("/task/" + strconv.Itoa(taskId) + "/complete").Expect().Status(200)
 
 	// Verify the task is marked as completed
 	e.GET("/task/" + strconv.Itoa(taskId)).Expect().Status(404)
-
-	// Verify the allowances are updated for user 1
-	allowances := e.GET("/user/1/allowance").Expect().Status(200).JSON().Array()
-	allowances.Length().IsEqual(2)
-	allowances.Value(0).Object().Value("id").Number().IsEqual(0)
-	allowances.Value(0).Object().Value("progress").Number().IsEqual(101)
-	allowances.Value(1).Object().Value("id").Number().IsEqual(1)
-	allowances.Value(1).Object().Value("progress").Number().IsEqual(0)
-}
-
-func TestCompleteTaskInvalidId(t *testing.T) {
-	e := startServer(t)
-	e.POST("/task/999/complete").Expect().Status(404)
-}
-
-func TestCompleteAllowance(t *testing.T) {
-	e := startServer(t)
-	createTestTaskWithAmount(e, 100)
-	createTestAllowance(e, "Test Allowance 1", 100, 50)
-
-	// Complete the task
-	e.POST("/task/1/complete").Expect().Status(200)
-
-	// Complete allowance goal
-	e.POST("/user/1/allowance/1/complete").Expect().Status(200)
-
-	// Verify the allowance no longer exists
-	e.GET("/user/1/allowance/1").Expect().Status(404)
-
-	// Verify history is updated
-	history := e.GET("/user/1/history").Expect().Status(200).JSON().Array()
-	history.Length().IsEqual(2)
-	history.Value(0).Object().Value("allowance").Number().IsEqual(100)
-	history.Value(0).Object().Value("timestamp").String().AsDateTime().InRange(getDelta(time.Now(), 2.0))
-	history.Value(1).Object().Value("allowance").Number().IsEqual(-100)
-	history.Value(1).Object().Value("timestamp").String().AsDateTime().InRange(getDelta(time.Now(), 2.0))
-}
-
-func TestCompleteAllowanceInvalidUserId(t *testing.T) {
-	e := startServer(t)
-	e.POST("/user/999/allowance/1/complete").Expect().Status(404)
-}
-
-func TestCompleteAllowanceInvalidAllowanceId(t *testing.T) {
-	e := startServer(t)
-	e.POST("/user/1/allowance/999/complete").Expect().Status(404)
-}
-
-func TestPutBulkAllowance(t *testing.T) {
-	e := startServer(t)
-
-	createTestAllowance(e, "Test Allowance 1", 1000, 1)
-	createTestAllowance(e, "Test Allowance 2", 1000, 2)
-
-	// Bulk edit
-	request := []map[string]interface{}{
-		{
-			"id":     1,
-			"weight": 5,
-		},
-		{
-			"id":     0,
-			"weight": 99,
-		},
-		{
-			"id":     2,
-			"weight": 10,
-		},
-	}
-	e.PUT("/user/1/allowance").WithJSON(request).Expect().Status(200)
-
 	// Verify the allowances are updated
 	allowances := e.GET("/user/1/allowance").Expect().Status(200).JSON().Array()
 	allowances.Length().IsEqual(3)
 	allowances.Value(0).Object().Value("id").Number().IsEqual(0)
-	allowances.Value(0).Object().Value("weight").Number().IsEqual(99)
+	allowances.Value(0).Object().Value("progress").Number().IsEqual(50)
-	allowances.Value(1).Object().Value("id").Number().IsEqual(1)
+	allowances.Value(0).Object().Value("id").Number().IsEqual(1)
-	allowances.Value(1).Object().Value("weight").Number().IsEqual(5)
+	allowances.Value(1).Object().Value("progress").Number().IsEqual(25)
-	allowances.Value(2).Object().Value("id").Number().IsEqual(2)
+	allowances.Value(0).Object().Value("id").Number().IsEqual(2)
-	allowances.Value(2).Object().Value("weight").Number().IsEqual(10)
+	allowances.Value(2).Object().Value("progress").Number().IsEqual(26)
 }
 
 func getDelta(base time.Time, delta float64) (time.Time, time.Time) {
@@ -695,15 +571,3 @@ func getDelta(base time.Time, delta float64) (time.Time, time.Time) {
 	end := base.Add(time.Duration(delta) * time.Second)
 	return start, end
 }
-
-func createTestAllowance(e *httpexpect.Expect, name string, target int, weight float64) {
-	e.POST("/user/1/allowance").WithJSON(CreateAllowanceRequest{
-		Name:   name,
-		Target: target,
-		Weight: weight,
-	}).Expect().Status(201)
-}
-
-func createTestTask(e *httpexpect.Expect) int {
-	return createTestTaskWithAmount(e, 100)
-}

backend/db.go

@@ -95,19 +95,11 @@ func (db *Db) GetUserAllowances(userId int) ([]Allowance, error) {
 
 func (db *Db) GetUserAllowanceById(userId int, allowanceId int) (*Allowance, error) {
 	allowance := &Allowance{}
-	if allowanceId == 0 {
+	err := db.db.Query("select id, name, target, balance, weight from allowances where user_id = ? and id = ?").
-		err := db.db.Query("select balance, weight from users where id = ?").
+		Bind(userId, allowanceId).
-			Bind(userId).ScanSingle(&allowance.Progress, &allowance.Weight)
+		ScanSingle(&allowance.ID, &allowance.Name, &allowance.Target, &allowance.Progress, &allowance.Weight)
-		if err != nil {
+	if err != nil {
-			return nil, err
+		return nil, err
-		}
-	} else {
-		err := db.db.Query("select id, name, target, balance, weight from allowances where user_id = ? and id = ?").
-			Bind(userId, allowanceId).
-			ScanSingle(&allowance.ID, &allowance.Name, &allowance.Target, &allowance.Progress, &allowance.Weight)
-		if err != nil {
-			return nil, err
-		}
 	}
 	return allowance, nil
 }
@@ -175,39 +167,6 @@ func (db *Db) DeleteAllowance(userId int, allowanceId int) error {
 	return nil
 }
 
-func (db *Db) CompleteAllowance(userId int, allowanceId int) error {
-	tx, err := db.db.Begin()
-	if err != nil {
-		return err
-	}
-	defer tx.MustRollback()
-
-	// Get the cost of the allowance
-	var cost int
-	err = tx.Query("select balance from allowances where id = ? and user_id = ?").
-		Bind(allowanceId, userId).ScanSingle(&cost)
-	if err != nil {
-		return err
-	}
-
-	// Delete the allowance
-	err = tx.Query("delete from allowances where id = ? and user_id = ?").
-		Bind(allowanceId, userId).Exec()
-	if err != nil {
-		return err
-	}
-
-	// Add a history entry
-	err = tx.Query("insert into history (user_id, timestamp, amount) values (?, ?, ?)").
-		Bind(userId, time.Now().Unix(), -cost).
-		Exec()
-	if err != nil {
-		return err
-	}
-
-	return tx.Commit()
-}
-
 func (db *Db) UpdateUserAllowance(userId int, allowance *UpdateAllowanceRequest) error {
 	tx, err := db.db.Begin()
 	if err != nil {
@@ -251,31 +210,6 @@ func (db *Db) UpdateAllowance(userId int, allowanceId int, allowance *UpdateAllo
 	return tx.Commit()
 }
 
-func (db *Db) BulkUpdateAllowance(userId int, allowances []BulkUpdateAllowanceRequest) error {
-	tx, err := db.db.Begin()
-	if err != nil {
-		return err
-	}
-	defer tx.MustRollback()
-
-	for _, allowance := range allowances {
-		if allowance.ID == 0 {
-			err = tx.Query("update users set weight=? where id = ?").
-				Bind(allowance.Weight, userId).
-				Exec()
-		} else {
-			err = tx.Query("update allowances set weight=? where id = ? and user_id = ?").
-				Bind(allowance.Weight, allowance.ID, userId).
-				Exec()
-		}
-		if err != nil {
-			return err
-		}
-	}
-
-	return tx.Commit()
-}
-
 func (db *Db) CreateTask(task *CreateTaskRequest) (int, error) {
 	tx, err := db.db.Begin()
 	if err != nil {
@@ -399,46 +333,30 @@ func (db *Db) CompleteTask(taskId int) error {
 			return err
 		}
 
-		// Add the history entry
-		err = tx.Query("insert into history (user_id, timestamp, amount) values (?, ?, ?)").
-			Bind(userId, time.Now().Unix(), reward).
-			Exec()
-		if err != nil {
-			return err
-		}
-
-		// Calculate the sums of all weights
 		var sumOfWeights float64
 		err = tx.Query("select sum(weight) from allowances where user_id = ? and weight > 0").Bind(userId).ScanSingle(&sumOfWeights)
-		sumOfWeights += userWeight
 
-		remainingReward := reward
+		for allowanceRow := range tx.Query("select id, weight from allowances where user_id = ? and weight > 0").Bind(userId).Range(&err) {
-
+			var allowanceId int
-		if sumOfWeights > 0 {
+			var allowanceWeight float64
-			// Distribute the reward to the allowances
+			err = allowanceRow.Scan(&allowanceId, &allowanceWeight)
-			for allowanceRow := range tx.Query("select id, weight from allowances where user_id = ? and weight > 0").Bind(userId).Range(&err) {
+			if err != nil {
-				var allowanceId int
+				return err
-				var allowanceWeight float64
-				err = allowanceRow.Scan(&allowanceId, &allowanceWeight)
-				if err != nil {
-					return err
-				}
-
-				// Calculate the amount to add to the allowance
-				amount := int((allowanceWeight / sumOfWeights) * float64(remainingReward))
-				sumOfWeights -= allowanceWeight
-				err = tx.Query("update allowances set balance = balance + ? where id = ? and user_id = ?").
-					Bind(amount, allowanceId, userId).Exec()
-				if err != nil {
-					return err
-				}
-				remainingReward -= amount
 			}
+
+			// Calculate the amount to add to the allowance
+			amount := int((allowanceWeight / sumOfWeights) * float64(reward))
+			err = tx.Query("update allowances set balance = balance + ? where id = ? and user_id = ?").
+				Bind(amount, allowanceId, userId).Exec()
+			if err != nil {
+				return err
+			}
+			reward -= amount
 		}
 
 		// Add the remaining reward to the user
 		err = tx.Query("update users set balance = balance + ? where id = ?").
-			Bind(remainingReward, userId).Exec()
+			Bind(reward, userId).Exec()
 		if err != nil {
 			return err
 		}

backend/dto.go

@@ -31,28 +31,23 @@ type Task struct {
 }
 
 type Allowance struct {
-	ID       int     `json:"id"`
+	ID       int    `json:"id"`
-	Name     string  `json:"name"`
+	Name     string `json:"name"`
-	Target   int     `json:"target"`
+	Target   int    `json:"target"`
-	Progress int     `json:"progress"`
+	Progress int    `json:"progress"`
-	Weight   float64 `json:"weight"`
+	Weight   int    `json:"weight"`
 }
 
 type CreateAllowanceRequest struct {
-	Name   string  `json:"name"`
+	Name   string `json:"name"`
-	Target int     `json:"target"`
+	Target int    `json:"target"`
-	Weight float64 `json:"weight"`
+	Weight int    `json:"weight"`
 }
 
 type UpdateAllowanceRequest struct {
-	Name   string  `json:"name"`
+	Name   string `json:"name"`
-	Target int     `json:"target"`
+	Target int    `json:"target"`
-	Weight float64 `json:"weight"`
+	Weight int    `json:"weight"`
-}
-
-type BulkUpdateAllowanceRequest struct {
-	ID     int     `json:"id"`
-	Weight float64 `json:"weight"`
 }
 
 type CreateGoalResponse struct {

backend/go.mod

@@ -3,7 +3,7 @@ module allowance_planner
 go 1.24.2
 
 require (
-	gitea.seeseepuff.be/seeseemelk/mysqlite v0.14.0
+	gitea.seeseepuff.be/seeseemelk/mysqlite v0.13.0
 	github.com/gavv/httpexpect/v2 v2.17.0
 	github.com/gin-contrib/cors v1.7.5
 	github.com/gin-gonic/gin v1.10.0
@@ -68,9 +68,9 @@ require (
 	gopkg.in/fsnotify.v1 v1.4.7 // indirect
 	gopkg.in/tomb.v1 v1.0.0-20141024135613-dd632973f1e7 // indirect
 	gopkg.in/yaml.v3 v3.0.1 // indirect
-	modernc.org/libc v1.65.7 // indirect
+	modernc.org/libc v1.65.6 // indirect
 	modernc.org/mathutil v1.7.1 // indirect
-	modernc.org/memory v1.11.0 // indirect
+	modernc.org/memory v1.10.0 // indirect
 	modernc.org/sqlite v1.37.0 // indirect
 	moul.io/http2curl/v2 v2.3.0 // indirect
 	zombiezen.com/go/sqlite v1.4.0 // indirect

backend/go.sum

@@ -2,8 +2,6 @@ gitea.seeseepuff.be/seeseemelk/mysqlite v0.12.0 h1:kl0VFgvm52UKxJhZpf1hvucxZdOoX
 gitea.seeseepuff.be/seeseemelk/mysqlite v0.12.0/go.mod h1:cgswydOxJjMlNwfcBIXnKjr47LwXnMT9BInkiHb0tXE=
 gitea.seeseepuff.be/seeseemelk/mysqlite v0.13.0 h1:nqSXu5i5fHB1rrx/kfi8Phn/J6eFa2yh02FiGc9U1yg=
 gitea.seeseepuff.be/seeseemelk/mysqlite v0.13.0/go.mod h1:cgswydOxJjMlNwfcBIXnKjr47LwXnMT9BInkiHb0tXE=
-gitea.seeseepuff.be/seeseemelk/mysqlite v0.14.0 h1:aRItVfUj48fBmuec7rm/jY9KCfvHW2VzJfItVk4t8sw=
-gitea.seeseepuff.be/seeseemelk/mysqlite v0.14.0/go.mod h1:cgswydOxJjMlNwfcBIXnKjr47LwXnMT9BInkiHb0tXE=
 github.com/TylerBrock/colorjson v0.0.0-20200706003622-8a50f05110d2 h1:ZBbLwSJqkHBuFDA6DUhhse0IGJ7T5bemHyNILUjvOq4=
 github.com/TylerBrock/colorjson v0.0.0-20200706003622-8a50f05110d2/go.mod h1:VSw57q4QFiWDbRnjdX8Cb3Ow0SFncRw+bA/ofY6Q83w=
 github.com/ajg/form v1.5.1 h1:t9c7v8JUKu/XxOGBU0yjNpaMloxGEJhUkqFRq0ibGeU=
@@ -220,14 +218,10 @@ modernc.org/gc/v2 v2.6.5 h1:nyqdV8q46KvTpZlsw66kWqwXRHdjIlJOhG6kxiV/9xI=
 modernc.org/gc/v2 v2.6.5/go.mod h1:YgIahr1ypgfe7chRuJi2gD7DBQiKSLMPgBQe9oIiito=
 modernc.org/libc v1.65.6 h1:OhJUhmuJ6MVZdqL5qmnd0/my46DKGFhSX4WOR7ijfyE=
 modernc.org/libc v1.65.6/go.mod h1:MOiGAM9lrMBT9L8xT1nO41qYl5eg9gCp9/kWhz5L7WA=
-modernc.org/libc v1.65.7 h1:Ia9Z4yzZtWNtUIuiPuQ7Qf7kxYrxP1/jeHZzG8bFu00=
-modernc.org/libc v1.65.7/go.mod h1:011EQibzzio/VX3ygj1qGFt5kMjP0lHb0qCW5/D/pQU=
 modernc.org/mathutil v1.7.1 h1:GCZVGXdaN8gTqB1Mf/usp1Y/hSqgI2vAGGP4jZMCxOU=
 modernc.org/mathutil v1.7.1/go.mod h1:4p5IwJITfppl0G4sUEDtCr4DthTaT47/N3aT6MhfgJg=
 modernc.org/memory v1.10.0 h1:fzumd51yQ1DxcOxSO+S6X7+QTuVU+n8/Aj7swYjFfC4=
 modernc.org/memory v1.10.0/go.mod h1:/JP4VbVC+K5sU2wZi9bHoq2MAkCnrt2r98UGeSK7Mjw=
-modernc.org/memory v1.11.0 h1:o4QC8aMQzmcwCK3t3Ux/ZHmwFPzE6hf2Y5LbkRs+hbI=
-modernc.org/memory v1.11.0/go.mod h1:/JP4VbVC+K5sU2wZi9bHoq2MAkCnrt2r98UGeSK7Mjw=
 modernc.org/opt v0.1.4 h1:2kNGMRiUjrp4LcaPuLY2PzUfqM/w9N23quVwhKt5Qm8=
 modernc.org/opt v0.1.4/go.mod h1:03fq9lsNfvkYSfxrfUhZCWPk1lm4cq4N+Bh//bEtgns=
 modernc.org/sortutil v1.2.1 h1:+xyoGf15mM3NMlPDnFqrteY07klSFxLElE2PVuWIJ7w=

backend/main.go

@@ -189,44 +189,6 @@ func createUserAllowance(c *gin.Context) {
 	c.IndentedJSON(http.StatusCreated, response)
 }
 
-func bulkPutUserAllowance(c *gin.Context) {
-	userIdStr := c.Param("userId")
-
-	userId, err := strconv.Atoi(userIdStr)
-	if err != nil {
-		log.Printf(ErrInvalidUserID+": %v", err)
-		c.JSON(http.StatusBadRequest, gin.H{"error": ErrInvalidUserID})
-		return
-	}
-
-	exists, err := db.UserExists(userId)
-	if err != nil {
-		log.Printf(ErrCheckingUserExist, err)
-		c.JSON(http.StatusInternalServerError, gin.H{"error": ErrInternalServerError})
-		return
-	}
-	if !exists {
-		c.JSON(http.StatusNotFound, gin.H{"error": ErrUserNotFound})
-		return
-	}
-
-	var allowanceRequest []BulkUpdateAllowanceRequest
-	if err := c.ShouldBindJSON(&allowanceRequest); err != nil {
-		log.Printf("Error parsing request body: %v", err)
-		c.JSON(http.StatusBadRequest, gin.H{"error": "Invalid request body"})
-		return
-	}
-
-	err = db.BulkUpdateAllowance(userId, allowanceRequest)
-	if err != nil {
-		log.Printf("Error updating allowance: %v", err)
-		c.JSON(http.StatusInternalServerError, gin.H{"error": ErrInternalServerError})
-		return
-	}
-
-	c.IndentedJSON(http.StatusOK, gin.H{"message": "Allowance updated successfully"})
-}
-
 func deleteUserAllowance(c *gin.Context) {
 	userIdStr := c.Param("userId")
 	allowanceIdStr := c.Param("allowanceId")
@@ -245,11 +207,6 @@ func deleteUserAllowance(c *gin.Context) {
 		return
 	}
 
-	if allowanceId == 0 {
-		c.JSON(http.StatusBadRequest, gin.H{"error": "Allowance id zero cannot be deleted"})
-		return
-	}
-
 	exists, err := db.UserExists(userId)
 	if err != nil {
 		log.Printf(ErrCheckingUserExist, err)
@@ -325,49 +282,6 @@ func putUserAllowance(c *gin.Context) {
 	c.IndentedJSON(http.StatusOK, gin.H{"message": "Allowance updated successfully"})
 }
 
-func completeAllowance(c *gin.Context) {
-	userIdStr := c.Param("userId")
-	allowanceIdStr := c.Param("allowanceId")
-
-	userId, err := strconv.Atoi(userIdStr)
-	if err != nil {
-		log.Printf(ErrInvalidUserID+": %v", err)
-		c.JSON(http.StatusBadRequest, gin.H{"error": ErrInvalidUserID})
-		return
-	}
-
-	allowanceId, err := strconv.Atoi(allowanceIdStr)
-	if err != nil {
-		log.Printf("Invalid allowance ID: %v", err)
-		c.JSON(http.StatusBadRequest, gin.H{"error": "Invalid allowance ID"})
-		return
-	}
-
-	exists, err := db.UserExists(userId)
-	if err != nil {
-		log.Printf(ErrCheckingUserExist, err)
-		c.JSON(http.StatusInternalServerError, gin.H{"error": ErrInternalServerError})
-		return
-	}
-	if !exists {
-		c.JSON(http.StatusNotFound, gin.H{"error": ErrUserNotFound})
-		return
-	}
-
-	err = db.CompleteAllowance(userId, allowanceId)
-	if errors.Is(err, mysqlite.ErrNoRows) {
-		c.JSON(http.StatusNotFound, gin.H{"error": "Allowance not found"})
-		return
-	}
-	if err != nil {
-		log.Printf("Error completing allowance: %v", err)
-		c.JSON(http.StatusInternalServerError, gin.H{"error": ErrInternalServerError})
-		return
-	}
-
-	c.IndentedJSON(http.StatusOK, gin.H{"message": "Allowance completed successfully"})
-}
-
 func createTask(c *gin.Context) {
 	var taskRequest CreateTaskRequest
 	if err := c.ShouldBindJSON(&taskRequest); err != nil {
@@ -596,11 +510,9 @@ func start(ctx context.Context, config *ServerConfig) {
 	router.GET("/api/user/:userId/history", getHistory)
 	router.GET("/api/user/:userId/allowance", getUserAllowance)
 	router.POST("/api/user/:userId/allowance", createUserAllowance)
-	router.PUT("/api/user/:userId/allowance", bulkPutUserAllowance)
 	router.GET("/api/user/:userId/allowance/:allowanceId", getUserAllowanceById)
 	router.DELETE("/api/user/:userId/allowance/:allowanceId", deleteUserAllowance)
 	router.PUT("/api/user/:userId/allowance/:allowanceId", putUserAllowance)
-	router.POST("/api/user/:userId/allowance/:allowanceId/complete", completeAllowance)
 	router.POST("/api/tasks", createTask)
 	router.GET("/api/tasks", getTasks)
 	router.GET("/api/task/:taskId", getTask)
@@ -641,9 +553,5 @@ func main() {
 		Datasource: os.Getenv("DB_PATH"),
 		Addr:       ":8080",
 	}
-	if config.Datasource == "" {
-		config.Datasource = "allowance_planner.db3"
-		log.Printf("Warning: No DB_PATH set, using default of %s", config.Datasource)
-	}
 	start(context.Background(), &config)
 }

backend/migrations/1_initial.sql

@@ -2,7 +2,7 @@ create table users
 (
     id   integer primary key,
     name text not null,
-    weight real not null default 0.0,
+    weight real not null default 1.0,
     balance integer not null default 0
 ) strict;