Improve compatibility with ancient browsers

backend/api_test.go

@@ -437,9 +437,9 @@ func TestPutTaskInvalidTaskId(t *testing.T) {
 func TestPostHistory(t *testing.T) {
 	e := startServer(t)
 
-	e.POST("/user/1/history").WithJSON(PostHistory{Allowance: 100, Description: "Add a 100"}).Expect().Status(200)
+	e.POST("/user/1/history").WithJSON(PostHistory{Allowance: 100}).Expect().Status(200)
-	e.POST("/user/1/history").WithJSON(PostHistory{Allowance: 20, Description: "Lolol"}).Expect().Status(200)
+	e.POST("/user/1/history").WithJSON(PostHistory{Allowance: 20}).Expect().Status(200)
-	e.POST("/user/1/history").WithJSON(PostHistory{Allowance: -10, Description: "Subtracting"}).Expect().Status(200)
+	e.POST("/user/1/history").WithJSON(PostHistory{Allowance: -10}).Expect().Status(200)
 
 	response := e.GET("/user/1").Expect().Status(200).JSON().Object()
 	response.Value("allowance").Number().IsEqual(100 + 20 - 10)
@@ -448,36 +448,23 @@ func TestPostHistory(t *testing.T) {
 func TestPostHistoryInvalidUserId(t *testing.T) {
 	e := startServer(t)
 
-	e.POST("/user/999/history").WithJSON(PostHistory{Allowance: 100, Description: "Good"}).Expect().
+	e.POST("/user/999/history").WithJSON(PostHistory{Allowance: 100}).Expect().
 		Status(404)
 }
 
-func TestPostHistoryInvalidDescription(t *testing.T) {
-	e := startServer(t)
-
-	e.POST("/user/1/history").WithJSON(PostHistory{Allowance: 100}).Expect().
-		Status(400)
-}
-
 func TestGetHistory(t *testing.T) {
 	e := startServer(t)
 
-	e.POST("/user/1/history").WithJSON(PostHistory{Allowance: 100, Description: "Add 100"}).Expect().Status(200)
+	e.POST("/user/1/history").WithJSON(PostHistory{Allowance: 100}).Expect().Status(200)
-	e.POST("/user/1/history").WithJSON(PostHistory{Allowance: 20, Description: "Add 20"}).Expect().Status(200)
+	e.POST("/user/1/history").WithJSON(PostHistory{Allowance: 20}).Expect().Status(200)
-	e.POST("/user/1/history").WithJSON(PostHistory{Allowance: -10, Description: "Subtract 10"}).Expect().Status(200)
+	e.POST("/user/1/history").WithJSON(PostHistory{Allowance: -10}).Expect().Status(200)
 
 	response := e.GET("/user/1/history").Expect().Status(200).JSON().Array()
 	response.Length().IsEqual(3)
-	response.Value(0).Object().Length().IsEqual(3)
 	response.Value(0).Object().Value("allowance").Number().IsEqual(100)
 	response.Value(0).Object().Value("timestamp").String().AsDateTime().InRange(getDelta(time.Now(), 2.0))
-	response.Value(0).Object().Value("description").String().IsEqual("Add 100")
-
 	response.Value(1).Object().Value("allowance").Number().IsEqual(20)
-	response.Value(1).Object().Value("description").String().IsEqual("Add 20")
-
 	response.Value(2).Object().Value("allowance").Number().IsEqual(-10)
-	response.Value(2).Object().Value("description").String().IsEqual("Subtract 10")
 }
 
 func TestGetUserAllowanceById(t *testing.T) {
@@ -703,15 +690,10 @@ func TestCompleteAllowance(t *testing.T) {
 	// Verify history is updated
 	history := e.GET("/user/1/history").Expect().Status(200).JSON().Array()
 	history.Length().IsEqual(2)
-	history.Value(0).Object().Length().IsEqual(3)
 	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(0).Object().Value("description").String().IsEqual("Task completed: Test Task")
-
-	history.Value(1).Object().Length().IsEqual(3)
 	history.Value(1).Object().Value("allowance").Number().IsEqual(-100)
 	history.Value(1).Object().Value("timestamp").String().AsDateTime().InRange(getDelta(time.Now(), 2.0))
-	history.Value(1).Object().Value("description").String().IsEqual("Allowance completed: Test Allowance 1")
 }
 
 func TestCompleteAllowanceInvalidUserId(t *testing.T) {
@@ -758,145 +740,6 @@ func TestPutBulkAllowance(t *testing.T) {
 	allowances.Value(2).Object().Value("weight").Number().IsEqual(10)
 }
 
-func TestAddAllowanceSimple(t *testing.T) {
-	e := startServer(t)
-
-	createTestAllowance(e, "Test Allowance 1", 1000, 1)
-
-	request := map[string]interface{}{
-		"amount":      10,
-		"description": "Added to allowance 1",
-	}
-	e.POST("/user/1/allowance/1/add").WithJSON(request).Expect().Status(200)
-
-	// Verify the allowance is updated
-	allowances := e.GET("/user/1/allowance").Expect().Status(200).JSON().Array()
-	allowances.Value(1).Object().Value("id").Number().IsEqual(1)
-	allowances.Value(1).Object().Value("progress").Number().InDelta(10.0, 0.01)
-
-	// Verify the history is updated
-	history := e.GET("/user/1/history").Expect().Status(200).JSON().Array()
-	history.Length().IsEqual(1)
-	history.Value(0).Object().Value("allowance").Number().InDelta(10.0, 0.01)
-	history.Value(0).Object().Value("timestamp").String().AsDateTime().InRange(getDelta(time.Now(), 2.0))
-	history.Value(0).Object().Value("description").String().IsEqual("Added to allowance 1")
-}
-
-func TestAddAllowanceWithSpillage(t *testing.T) {
-	e := startServer(t)
-
-	createTestAllowance(e, "Test Allowance 1", 5, 1)
-	createTestAllowance(e, "Test Allowance 2", 5, 1)
-	e.PUT("/user/1/allowance/0").WithJSON(UpdateAllowanceRequest{Weight: 1}).Expect().Status(200)
-
-	request := map[string]interface{}{
-		"amount":      10,
-		"description": "Added to allowance 1",
-	}
-	e.POST("/user/1/allowance/1/add").WithJSON(request).Expect().Status(200)
-
-	// Verify the allowance is updated
-	allowances := e.GET("/user/1/allowance").Expect().Status(200).JSON().Array()
-	allowances.Value(1).Object().Value("id").Number().IsEqual(1)
-	allowances.Value(1).Object().Value("progress").Number().InDelta(5.0, 0.01)
-
-	allowances.Value(2).Object().Value("id").Number().IsEqual(2)
-	allowances.Value(2).Object().Value("progress").Number().InDelta(2.5, 0.01)
-
-	allowances.Value(0).Object().Value("id").Number().IsEqual(0)
-	allowances.Value(0).Object().Value("progress").Number().InDelta(2.5, 0.01)
-
-	// Verify the history is updated
-	history := e.GET("/user/1/history").Expect().Status(200).JSON().Array()
-	history.Length().IsEqual(1)
-	history.Value(0).Object().Value("allowance").Number().InDelta(10.0, 0.01)
-	history.Value(0).Object().Value("timestamp").String().AsDateTime().InRange(getDelta(time.Now(), 2.0))
-	history.Value(0).Object().Value("description").String().IsEqual("Added to allowance 1")
-}
-
-func TestAddAllowanceIdZero(t *testing.T) {
-	e := startServer(t)
-
-	createTestAllowance(e, "Test Allowance 1", 1000, 1)
-
-	request := map[string]interface{}{
-		"amount":      10,
-		"description": "Added to allowance 1",
-	}
-	e.POST("/user/1/allowance/0/add").WithJSON(request).Expect().Status(200)
-
-	// Verify the allowance is updated
-	allowances := e.GET("/user/1/allowance").Expect().Status(200).JSON().Array()
-	allowances.Value(0).Object().Value("id").Number().IsEqual(0)
-	allowances.Value(0).Object().Value("progress").Number().InDelta(10.0, 0.01)
-
-	// Verify the history is updated
-	history := e.GET("/user/1/history").Expect().Status(200).JSON().Array()
-	history.Length().IsEqual(1)
-	history.Value(0).Object().Value("allowance").Number().InDelta(10.0, 0.01)
-	history.Value(0).Object().Value("timestamp").String().AsDateTime().InRange(getDelta(time.Now(), 2.0))
-	history.Value(0).Object().Value("description").String().IsEqual("Added to allowance 1")
-}
-
-func TestSubtractAllowanceSimple(t *testing.T) {
-	e := startServer(t)
-
-	createTestAllowance(e, "Test Allowance 1", 1000, 1)
-
-	request := map[string]interface{}{
-		"amount":      10,
-		"description": "Added to allowance 1",
-	}
-	e.POST("/user/1/allowance/1/add").WithJSON(request).Expect().Status(200)
-	request["amount"] = -2.5
-	e.POST("/user/1/allowance/1/add").WithJSON(request).Expect().Status(200)
-
-	// Verify the allowance is updated
-	allowances := e.GET("/user/1/allowance").Expect().Status(200).JSON().Array()
-	allowances.Value(1).Object().Value("id").Number().IsEqual(1)
-	allowances.Value(1).Object().Value("progress").Number().InDelta(7.5, 0.01)
-
-	// Verify the 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().InDelta(10.0, 0.01)
-	history.Value(0).Object().Value("timestamp").String().AsDateTime().InRange(getDelta(time.Now(), 2.0))
-	history.Value(0).Object().Value("description").String().IsEqual("Added to allowance 1")
-
-	history.Value(1).Object().Value("allowance").Number().InDelta(-2.5, 0.01)
-	history.Value(1).Object().Value("timestamp").String().AsDateTime().InRange(getDelta(time.Now(), 2.0))
-	history.Value(1).Object().Value("description").String().IsEqual("Added to allowance 1")
-}
-
-func TestSubtractllowanceIdZero(t *testing.T) {
-	e := startServer(t)
-
-	createTestAllowance(e, "Test Allowance 1", 1000, 1)
-
-	request := map[string]interface{}{
-		"amount":      10,
-		"description": "Added to allowance 1",
-	}
-	e.POST("/user/1/allowance/0/add").WithJSON(request).Expect().Status(200)
-	request["amount"] = -2.5
-	e.POST("/user/1/allowance/0/add").WithJSON(request).Expect().Status(200)
-
-	// Verify the allowance is updated
-	allowances := e.GET("/user/1/allowance").Expect().Status(200).JSON().Array()
-	allowances.Value(0).Object().Value("id").Number().IsEqual(0)
-	allowances.Value(0).Object().Value("progress").Number().InDelta(7.5, 0.01)
-
-	// Verify the 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().InDelta(10.0, 0.01)
-	history.Value(0).Object().Value("timestamp").String().AsDateTime().InRange(getDelta(time.Now(), 2.0))
-	history.Value(0).Object().Value("description").String().IsEqual("Added to allowance 1")
-
-	history.Value(1).Object().Value("allowance").Number().InDelta(-2.5, 0.01)
-	history.Value(1).Object().Value("description").String().IsEqual("Added to allowance 1")
-}
-
 func getDelta(base time.Time, delta float64) (time.Time, time.Time) {
 	start := base.Add(-time.Duration(delta) * time.Second)
 	end := base.Add(time.Duration(delta) * time.Second)

backend/db.go

@@ -2,7 +2,6 @@ package main
 
 import (
 	"errors"
-	"fmt"
 	"log"
 	"math"
 	"time"
@@ -207,9 +206,8 @@ func (db *Db) CompleteAllowance(userId int, allowanceId int) error {
 
 	// Get the cost of the allowance
 	var cost int
-	var allowanceName string
+	err = tx.Query("select balance from allowances where id = ? and user_id = ?").
-	err = tx.Query("select balance, name from allowances where id = ? and user_id = ?").
+		Bind(allowanceId, userId).ScanSingle(&cost)
-		Bind(allowanceId, userId).ScanSingle(&cost, &allowanceName)
 	if err != nil {
 		return err
 	}
@@ -222,8 +220,8 @@ func (db *Db) CompleteAllowance(userId int, allowanceId int) error {
 	}
 
 	// Add a history entry
-	err = tx.Query("insert into history (user_id, timestamp, amount, description) values (?, ?, ?, ?)").
+	err = tx.Query("insert into history (user_id, timestamp, amount) values (?, ?, ?)").
-		Bind(userId, time.Now().Unix(), -cost, fmt.Sprintf("Allowance completed: %s", allowanceName)).
+		Bind(userId, time.Now().Unix(), -cost).
 		Exec()
 	if err != nil {
 		return err
@@ -422,28 +420,63 @@ func (db *Db) CompleteTask(taskId int) error {
 	defer tx.MustRollback()
 
 	var reward int
-	var rewardName string
+	err = tx.Query("select reward from tasks where id = ?").Bind(taskId).ScanSingle(&reward)
-	err = tx.Query("select reward, name from tasks where id = ?").Bind(taskId).ScanSingle(&reward, &rewardName)
 	if err != nil {
 		return err
 	}
 
-	for userRow := range tx.Query("select id from users").Range(&err) {
+	for userRow := range tx.Query("select id, weight from users").Range(&err) {
 		var userId int
-		err = userRow.Scan(&userId)
+		var userWeight float64
+		err = userRow.Scan(&userId, &userWeight)
 		if err != nil {
 			return err
 		}
 
 		// Add the history entry
-		err = tx.Query("insert into history (user_id, timestamp, amount, description) values (?, ?, ?, ?)").
+		err = tx.Query("insert into history (user_id, timestamp, amount) values (?, ?, ?)").
-			Bind(userId, time.Now().Unix(), reward, fmt.Sprintf("Task completed: %s", rewardName)).
+			Bind(userId, time.Now().Unix(), reward).
 			Exec()
 		if err != nil {
 			return err
 		}
 
-		err := db.addDistributedReward(tx, userId, reward)
+		// 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
+
+		if sumOfWeights > 0 {
+			// Distribute the reward to the allowances
+			for allowanceRow := range tx.Query("select id, weight, target, balance from allowances where user_id = ? and weight > 0 order by (target - balance) asc").Bind(userId).Range(&err) {
+				var allowanceId, allowanceTarget, allowanceBalance int
+				var allowanceWeight float64
+				err = allowanceRow.Scan(&allowanceId, &allowanceWeight, &allowanceTarget, &allowanceBalance)
+				if err != nil {
+					return err
+				}
+
+				// Calculate the amount to add to the allowance
+				amount := int((allowanceWeight / sumOfWeights) * float64(remainingReward))
+				if allowanceBalance+amount > allowanceTarget {
+					// If the amount reaches past the target, set it to the target
+					amount = allowanceTarget - allowanceBalance
+				}
+				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
+			}
+		}
+
+		// Add the remaining reward to the user
+		err = tx.Query("update users set balance = balance + ? where id = ?").
+			Bind(remainingReward, userId).Exec()
 		if err != nil {
 			return err
 		}
@@ -458,52 +491,6 @@ func (db *Db) CompleteTask(taskId int) error {
 	return tx.Commit()
 }
 
-func (db *Db) addDistributedReward(tx *mysqlite.Tx, userId int, reward int) error {
-	var userWeight float64
-	err := tx.Query("select weight from users where id = ?").Bind(userId).ScanSingle(&userWeight)
-	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
-
-	if sumOfWeights > 0 {
-		// Distribute the reward to the allowances
-		for allowanceRow := range tx.Query("select id, weight, target, balance from allowances where user_id = ? and weight > 0 order by (target - balance) asc").Bind(userId).Range(&err) {
-			var allowanceId, allowanceTarget, allowanceBalance int
-			var allowanceWeight float64
-			err = allowanceRow.Scan(&allowanceId, &allowanceWeight, &allowanceTarget, &allowanceBalance)
-			if err != nil {
-				return err
-			}
-
-			// Calculate the amount to add to the allowance
-			amount := int((allowanceWeight / sumOfWeights) * float64(remainingReward))
-			if allowanceBalance+amount > allowanceTarget {
-				// If the amount reaches past the target, set it to the target
-				amount = allowanceTarget - allowanceBalance
-			}
-			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
-		}
-	}
-
-	// Add the remaining reward to the user
-	err = tx.Query("update users set balance = balance + ? where id = ?").
-		Bind(remainingReward, userId).Exec()
-	return err
-}
-
 func (db *Db) AddHistory(userId int, allowance *PostHistory) error {
 	tx, err := db.db.Begin()
 	if err != nil {
@@ -512,8 +499,8 @@ func (db *Db) AddHistory(userId int, allowance *PostHistory) error {
 	defer tx.MustRollback()
 
 	amount := int(math.Round(allowance.Allowance * 100.0))
-	err = tx.Query("insert into history (user_id, timestamp, amount, description) values (?, ?, ?, ?)").
+	err = tx.Query("insert into history (user_id, timestamp, amount) values (?, ?, ?)").
-		Bind(userId, time.Now().Unix(), amount, allowance.Description).
+		Bind(userId, time.Now().Unix(), amount).
 		Exec()
 	if err != nil {
 		return err
@@ -525,11 +512,11 @@ func (db *Db) GetHistory(userId int) ([]History, error) {
 	history := make([]History, 0)
 	var err error
 
-	for row := range db.db.Query("select amount, `timestamp`, description from history where user_id = ? order by `timestamp` desc").
+	for row := range db.db.Query("select amount, `timestamp` from history where user_id = ? order by `timestamp` desc").
 		Bind(userId).Range(&err) {
 		allowance := History{}
 		var timestamp, amount int64
-		err = row.Scan(&amount, &timestamp, &allowance.Description)
+		err = row.Scan(&amount, &timestamp)
 		if err != nil {
 			return nil, err
 		}
@@ -542,92 +529,3 @@ func (db *Db) GetHistory(userId int) ([]History, error) {
 	}
 	return history, nil
 }
-
-func (db *Db) AddAllowanceAmount(userId int, allowanceId int, request AddAllowanceAmountRequest) error {
-	tx, err := db.db.Begin()
-	if err != nil {
-		return err
-	}
-	defer tx.MustRollback()
-
-	// Convert amount to integer (cents)
-	remainingAmount := int(math.Round(request.Amount * 100))
-
-	// Insert history entry
-	err = tx.Query("insert into history (user_id, timestamp, amount, description) values (?, ?, ?, ?)").
-		Bind(userId, time.Now().Unix(), remainingAmount, request.Description).
-		Exec()
-	if err != nil {
-		return err
-	}
-
-	if allowanceId == 0 {
-		if remainingAmount < 0 {
-			var userBalance int
-			err = tx.Query("select balance from users where id = ?").
-				Bind(userId).ScanSingle(&userBalance)
-			if err != nil {
-				return err
-			}
-			if remainingAmount > userBalance {
-				return fmt.Errorf("cannot remove more than the current balance: %d", userBalance)
-			}
-		}
-		err = tx.Query("update users set balance = balance + ? where id = ?").
-			Bind(remainingAmount, userId).Exec()
-		if err != nil {
-			return err
-		}
-	} else if remainingAmount < 0 {
-		var progress int
-		err = tx.Query("select balance from allowances where id = ? and user_id = ?").
-			Bind(allowanceId, userId).ScanSingle(&progress)
-		if err != nil {
-			return err
-		}
-
-		if remainingAmount > progress {
-			return fmt.Errorf("cannot remove more than the current allowance balance: %d", progress)
-		}
-
-		err = tx.Query("update allowances set balance = balance + ? where id = ? and user_id = ?").
-			Bind(remainingAmount, allowanceId, userId).Exec()
-		if err != nil {
-			return err
-		}
-	} else {
-		// Fetch the target and progress of the specified allowance
-		var target, progress int
-		err = tx.Query("select target, balance from allowances where id = ? and user_id = ?").
-			Bind(allowanceId, userId).ScanSingle(&target, &progress)
-		if err != nil {
-			return err
-		}
-
-		// Calculate the amount to add to the current allowance
-		toAdd := remainingAmount
-		if progress+toAdd > target {
-			toAdd = target - progress
-		}
-		remainingAmount -= toAdd
-
-		// Update the current allowance
-		if toAdd > 0 {
-			err = tx.Query("update allowances set balance = balance + ? where id = ? and user_id = ?").
-				Bind(toAdd, allowanceId, userId).Exec()
-			if err != nil {
-				return err
-			}
-		}
-
-		// If there's remaining amount, distribute it to the user's allowances
-		if remainingAmount > 0 {
-			err = db.addDistributedReward(tx, userId, remainingAmount)
-			if err != nil {
-				return err
-			}
-		}
-	}
-
-	return tx.Commit()
-}

backend/dto.go

@@ -14,14 +14,12 @@ type UserWithAllowance struct {
 }
 
 type History struct {
-	Allowance   float64   `json:"allowance"`
+	Allowance float64   `json:"allowance"`
-	Timestamp   time.Time `json:"timestamp"`
+	Timestamp time.Time `json:"timestamp"`
-	Description string    `json:"description"`
 }
 
 type PostHistory struct {
-	Allowance   float64 `json:"allowance"`
+	Allowance float64 `json:"allowance"`
-	Description string  `json:"description"`
 }
 
 // Task represents a task in the system.
@@ -73,8 +71,3 @@ type CreateTaskRequest struct {
 type CreateTaskResponse struct {
 	ID int `json:"id"`
 }
-
-type AddAllowanceAmountRequest struct {
-	Amount      float64 `json:"amount"`
-	Description string  `json:"description"`
-}

backend/main.go

@@ -373,56 +373,6 @@ func completeAllowance(c *gin.Context) {
 	c.IndentedJSON(http.StatusOK, gin.H{"message": "Allowance completed successfully"})
 }
 
-func addToAllowance(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
-	}
-
-	var allowanceRequest AddAllowanceAmountRequest
-	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.AddAllowanceAmount(userId, allowanceId, allowanceRequest)
-	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 {
@@ -594,11 +544,6 @@ func postHistory(c *gin.Context) {
 		return
 	}
 
-	if historyRequest.Description == "" {
-		c.JSON(http.StatusBadRequest, gin.H{"error": "Description cannot be empty"})
-		return
-	}
-
 	exists, err := db.UserExists(userId)
 	if err != nil {
 		log.Printf(ErrCheckingUserExist, err)
@@ -666,7 +611,6 @@ func start(ctx context.Context, config *ServerConfig) {
 	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/user/:userId/allowance/:allowanceId/add", addToAllowance)
 	router.POST("/api/tasks", createTask)
 	router.GET("/api/tasks", getTasks)
 	router.GET("/api/task/:taskId", getTask)

backend/migrations/4_add_history_description.sql

@@ -1,2 +0,0 @@
-alter table history
-add column description text;

backend/web.go

@@ -28,13 +28,8 @@ func loadWebEndpoints(router *gin.Engine) {
 }
 
 func redirectToPage(c *gin.Context, page string) {
-	redirectToPageStatus(c, page, http.StatusSeeOther)
-}
-
-func redirectToPageStatus(c *gin.Context, page string, status int) {
 	scheme := c.Request.URL.Scheme
-	target := scheme + domain + page
+	c.Redirect(http.StatusSeeOther, scheme+domain+page)
-	c.Redirect(status, target)
 }
 
 func renderLogin(c *gin.Context) {
@@ -80,7 +75,7 @@ func renderCreateTask(c *gin.Context) {
 		return
 	}
 
-	redirectToPageStatus(c, "/", http.StatusFound)
+	c.Redirect(http.StatusFound, "/")
 }
 
 func renderCompleteTask(c *gin.Context) {
@@ -97,7 +92,7 @@ func renderCompleteTask(c *gin.Context) {
 		return
 	}
 
-	redirectToPageStatus(c, "/", http.StatusFound)
+	c.Redirect(http.StatusFound, "/")
 }
 
 func renderCreateAllowance(c *gin.Context) {
@@ -134,7 +129,7 @@ func renderCreateAllowance(c *gin.Context) {
 		return
 	}
 
-	redirectToPageStatus(c, "/", http.StatusFound)
+	c.Redirect(http.StatusFound, "/")
 }
 
 func renderCompleteAllowance(c *gin.Context) {
@@ -156,7 +151,7 @@ func renderCompleteAllowance(c *gin.Context) {
 		return
 	}
 
-	redirectToPageStatus(c, "/", http.StatusFound)
+	c.Redirect(http.StatusFound, "/")
 }
 
 func getCurrentUser(c *gin.Context) *int {
@@ -185,7 +180,7 @@ func getCurrentUser(c *gin.Context) *int {
 
 func unsetUserCookie(c *gin.Context) {
 	c.SetCookie("user", "", -1, "/", "localhost", false, true)
-	redirectToPageStatus(c, "/", http.StatusFound)
+	c.Redirect(http.StatusFound, "/")
 }
 
 func renderNoUser(c *gin.Context) {

backend/web.gohtml

@@ -3,11 +3,9 @@
 <head>
 	<title>Allowance Planner 2000</title>
 	<style>
-		<!--
 		tr:hover {
 			background-color: #f0f0f0;
 		}
-		-->
 	</style>
 </head>
 <body>
@@ -29,7 +27,7 @@
 	{{if ne .CurrentUser 0}}
 		<h2>Allowances</h2>
 		<form action="/createAllowance" method="post">
-			<table border=1>
+			<table border="1">
 				<thead>
 				<tr>
 					<th>Name</th>
@@ -45,7 +43,7 @@
 						<td></td>
 						<td><label><input type="number" name="target" placeholder="Target"></label></td>
 						<td><label><input type="number" name="weight" placeholder="Weight"></label></td>
-						<td><input type="submit" value="Create"></td>
+						<td><button>Create</button></td>
 					</tr>
 				{{range .Allowances}}
 					{{if eq .ID 0}}
@@ -105,7 +103,7 @@
 							<td><label><input type="text" name="name" placeholder="Name"></label></td>
 							<td></td>
 							<td><label><input type="number" name="reward" placeholder="Reward"></label></td>
-							<td><input type="submit" value="Create"></td>
+							<td><button>Create</button></td>
 						</tr>
 				</tbody>
 			</table>

frontend/allowance-planner-v2/android/app/src/main/res/mipmap-anydpi-v26/ic_launcher.xml

@@ -1,9 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
 <adaptive-icon xmlns:android="http://schemas.android.com/apk/res/android">
-    <background>
+    <background android:drawable="@color/ic_launcher_background"/>
-        <inset android:drawable="@mipmap/ic_launcher_background" android:inset="16.7%" />
+    <foreground android:drawable="@mipmap/ic_launcher_foreground"/>
-    </background>
-    <foreground>
-        <inset android:drawable="@mipmap/ic_launcher_foreground" android:inset="16.7%" />
-    </foreground>
 </adaptive-icon>

frontend/allowance-planner-v2/android/app/src/main/res/mipmap-anydpi-v26/ic_launcher_round.xml

@@ -1,9 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
 <adaptive-icon xmlns:android="http://schemas.android.com/apk/res/android">
-    <background>
+    <background android:drawable="@color/ic_launcher_background"/>
-        <inset android:drawable="@mipmap/ic_launcher_background" android:inset="16.7%" />
+    <foreground android:drawable="@mipmap/ic_launcher_foreground"/>
-    </background>
-    <foreground>
-        <inset android:drawable="@mipmap/ic_launcher_foreground" android:inset="16.7%" />
-    </foreground>
 </adaptive-icon>

frontend/allowance-planner-v2/android/app/src/main/res/values/strings.xml

@@ -1,7 +1,7 @@
 <?xml version='1.0' encoding='utf-8'?>
 <resources>
-    <string name="app_name">Allowance Planner V2</string>
+    <string name="app_name">allowance-planner-v2</string>
-    <string name="title_activity_main">Allowance Planner V2</string>
+    <string name="title_activity_main">allowance-planner-v2</string>
     <string name="package_name">io.ionic.starter</string>
     <string name="custom_url_scheme">io.ionic.starter</string>
 </resources>

frontend/allowance-planner-v2/capacitor.config.ts

@@ -2,7 +2,7 @@ import type { CapacitorConfig } from '@capacitor/cli';
 
 const config: CapacitorConfig = {
   appId: 'io.ionic.starter',
-  appName: 'Allowance Planner V2',
+  appName: 'allowance-planner-v2',
   webDir: 'www'
 };
 

frontend/allowance-planner-v2/package.json

@@ -47,7 +47,6 @@
     "@angular/cli": "^19.0.0",
     "@angular/compiler-cli": "^19.0.0",
     "@angular/language-service": "^19.0.0",
-    "@capacitor/assets": "^3.0.5",
     "@capacitor/cli": "7.2.0",
     "@ionic/angular-toolkit": "^12.0.0",
     "@types/jasmine": "~5.1.0",

frontend/allowance-planner-v2/src/app/app-routing.module.ts

@@ -11,6 +11,7 @@ const routes: Routes = [
     path: '',
     loadChildren: () => import('./pages/tabs/tabs.module').then(m => m.TabsPageModule)
   },
+
 ];
 @NgModule({
   imports: [

frontend/allowance-planner-v2/src/app/models/history.ts

@@ -1,5 +0,0 @@
-export interface History {
-    timestamp: string;
-    allowance: number;
-    description: string;
-}

frontend/allowance-planner-v2/src/app/pages/add-allowance/add-allowance-routing.module.ts

@@ -1,17 +0,0 @@
-import { NgModule } from '@angular/core';
-import { Routes, RouterModule } from '@angular/router';
-
-import { AddAllowancePage } from './add-allowance.page';
-
-const routes: Routes = [
-  {
-    path: '',
-    component: AddAllowancePage
-  }
-];
-
-@NgModule({
-  imports: [RouterModule.forChild(routes)],
-  exports: [RouterModule],
-})
-export class AddAllowancePageRoutingModule {}

frontend/allowance-planner-v2/src/app/pages/add-allowance/add-allowance.module.ts

@@ -1,23 +0,0 @@
-import { NgModule } from '@angular/core';
-import { CommonModule } from '@angular/common';
-import { FormsModule, ReactiveFormsModule } from '@angular/forms';
-
-import { IonicModule } from '@ionic/angular';
-
-import { AddAllowancePageRoutingModule } from './add-allowance-routing.module';
-
-import { AddAllowancePage } from './add-allowance.page';
-import { MatIconModule } from '@angular/material/icon';
-
-@NgModule({
-  imports: [
-    CommonModule,
-    FormsModule,
-    IonicModule,
-    AddAllowancePageRoutingModule,
-    ReactiveFormsModule,
-    MatIconModule
-  ],
-  declarations: [AddAllowancePage]
-})
-export class AddAllowancePageModule {}

frontend/allowance-planner-v2/src/app/pages/add-allowance/add-allowance.page.html

@@ -1,27 +0,0 @@
-<ion-header [translucent]="true">
-  <ion-toolbar>
-    <div class="toolbar">
-      <div class="icon" (click)="navigateBack()">
-        <mat-icon>arrow_back</mat-icon>
-      </div>
-      <ion-title *ngIf="isAddMode && goalId == 0">Add to Allowance</ion-title>
-      <ion-title *ngIf="isAddMode && goalId != 0">Add to Goal</ion-title>
-      <ion-title *ngIf="!isAddMode">Spend Allowance</ion-title>
-    </div>
-  </ion-toolbar>
-</ion-header>
-
-<ion-content [fullscreen]="true">
-  <form [formGroup]="form">
-    <label>Amount</label>
-    <input id="amount" type="number" placeholder="0.00" name="price" min="0" value="0" step="0.01" formControlName="amount"/>
-
-    <label>Description</label>
-    <input id="description" type="text" formControlName="description"/>
-
-    <button type="button" [disabled]="!form.valid" (click)="changeAllowance()">
-      <span *ngIf="isAddMode">Add</span>
-      <span *ngIf="!isAddMode">Spend</span>
-    </button>
-  </form>
-</ion-content>

frontend/allowance-planner-v2/src/app/pages/add-allowance/add-allowance.page.scss

@@ -1,49 +0,0 @@
-.toolbar {
-    display: flex;
-    align-items: center;
-}
-
-.icon {
-    margin-left: 5px;
-}
-
-form {
-    height: 100%;
-}
-
-form,
-.item {
-    display: flex;
-    flex-direction: column;
-    align-items: center;
-}
-
-input {
-    border: 1px solid var(--ion-color-primary);
-    border-radius: 5px;
-    width: 250px;
-    height: 40px;
-    padding-inline: 10px;
-    display: flex;
-    align-items: center;
-}
-
-label {
-    color: var(--ion-color-primary);
-    margin-top: 25px;
-    margin-bottom: 10px;
-}
-
-button {
-    background-color: var(--ion-color-primary);
-    border-radius: 5px;
-    color: white;
-    padding: 10px;
-    width: 250px;
-    margin-top: 100px;
-}
-
-button:disabled,
-button[disabled]{
-    opacity: 0.5;
-}

frontend/allowance-planner-v2/src/app/pages/add-allowance/add-allowance.page.spec.ts

@@ -1,17 +0,0 @@
-import { ComponentFixture, TestBed } from '@angular/core/testing';
-import { AddAllowancePage } from './add-allowance.page';
-
-describe('AddAllowancePage', () => {
-  let component: AddAllowancePage;
-  let fixture: ComponentFixture<AddAllowancePage>;
-
-  beforeEach(() => {
-    fixture = TestBed.createComponent(AddAllowancePage);
-    component = fixture.componentInstance;
-    fixture.detectChanges();
-  });
-
-  it('should create', () => {
-    expect(component).toBeTruthy();
-  });
-});

frontend/allowance-planner-v2/src/app/pages/add-allowance/add-allowance.page.ts

@@ -1,51 +0,0 @@
-import { Location } from '@angular/common';
-import { Component } from '@angular/core';
-import { FormBuilder, FormGroup, Validators } from '@angular/forms';
-import { ActivatedRoute, Router } from '@angular/router';
-import { AllowanceService } from 'src/app/services/allowance.service';
-
-@Component({
-  selector: 'app-add-allowance',
-  templateUrl: './add-allowance.page.html',
-  styleUrls: ['./add-allowance.page.scss'],
-  standalone: false,
-})
-export class AddAllowancePage {
-  public form: FormGroup;
-  public goalId: number;
-  public userId: number;
-  public isAddMode = true;
-// Marcus' first comment
-//            b    ........a`.OK  ø¶Ópppppppp--P09OP
-
-
-  constructor(
-    private allowanceService: AllowanceService,
-    private route: ActivatedRoute,
-    private formBuilder: FormBuilder,
-    private router: Router,
-    private location: Location
-  ) {
-    this.userId = this.route.snapshot.params['id'];
-    this.goalId = this.route.snapshot.params['goalId'];
-
-    this.form = this.formBuilder.group({
-      amount: ['', Validators.required],
-      description: ['', Validators.required]
-    });
-  }
-
-  changeAllowance() {
-    this.allowanceService.addOrSpendAllowance(
-      this.goalId,
-      this.userId,
-      this.form.value.amount,
-      this.form.value.description
-    );
-    this.router.navigate(['/tabs/allowance', this.userId]);
-  }
-
-  navigateBack() {
-    this.location.back();
-  }
-}

frontend/allowance-planner-v2/src/app/pages/add-allowance/spend-allowance-routing.module.ts

@@ -1,17 +0,0 @@
-import { NgModule } from '@angular/core';
-import { Routes, RouterModule } from '@angular/router';
-
-import { SpendllowancePage } from './spend-allowance.page';
-
-const routes: Routes = [
-    {
-        path: '',
-        component: SpendllowancePage
-    }
-];
-
-@NgModule({
-    imports: [RouterModule.forChild(routes)],
-    exports: [RouterModule],
-})
-export class SpendAllowancePageRoutingModule {}

frontend/allowance-planner-v2/src/app/pages/add-allowance/spend-allowance.module.ts

@@ -1,22 +0,0 @@
-import { NgModule } from '@angular/core';
-import { CommonModule } from '@angular/common';
-import { FormsModule, ReactiveFormsModule } from '@angular/forms';
-
-import { IonicModule } from '@ionic/angular';
-
-import { SpendAllowancePageRoutingModule } from './spend-allowance-routing.module';
-import { SpendllowancePage } from './spend-allowance.page';
-import { MatIconModule } from '@angular/material/icon';
-
-@NgModule({
-    imports: [
-        CommonModule,
-        FormsModule,
-        IonicModule,
-        SpendAllowancePageRoutingModule,
-        ReactiveFormsModule,
-        MatIconModule
-    ],
-    declarations: [SpendllowancePage]
-})
-export class SpendAllowancePageModule {}

frontend/allowance-planner-v2/src/app/pages/add-allowance/spend-allowance.page.ts

@@ -1,52 +0,0 @@
-import { Location } from '@angular/common';
-import { Component } from '@angular/core';
-import { FormBuilder, FormGroup, Validators } from '@angular/forms';
-import { ActivatedRoute, Router } from '@angular/router';
-import { AllowanceService } from 'src/app/services/allowance.service';
-
-@Component({
-    selector: 'app-spend-allowance',
-    templateUrl: './add-allowance.page.html',
-    styleUrls: ['./add-allowance.page.scss'],
-    standalone: false,
-})
-export class SpendllowancePage {
-    public form: FormGroup;
-    public goalId: number;
-    public userId: number;
-    public isAddMode = false;
-
-    constructor(
-        private allowanceService: AllowanceService,
-        private route: ActivatedRoute,
-        private formBuilder: FormBuilder,
-        private router: Router,
-        private location: Location
-    ) {
-        this.userId = this.route.snapshot.params['id'];
-        this.goalId = this.route.snapshot.params['goalId'];
-
-        this.form = this.formBuilder.group({
-        amount: ['', Validators.required],
-        description: ['', Validators.required]
-        });
-
-        this.allowanceService.getAllowanceById(this.goalId, this.userId).subscribe(allowance => {
-            this.form.controls['amount'].addValidators([Validators.max(allowance.progress)]);
-        }); 
-    }
-
-    changeAllowance() {
-        this.allowanceService.addOrSpendAllowance(
-        this.goalId,
-        this.userId,
-        -this.form.value.amount,
-        this.form.value.description
-        );
-        this.router.navigate(['/tabs/allowance', this.userId]);
-    }
-
-    navigateBack() {
-        this.location.back();
-    }
-}

frontend/allowance-planner-v2/src/app/pages/allowance/allowance-routing.module.ts

@@ -6,22 +6,6 @@ const routes: Routes = [
   {
     path: ':id',
     component: AllowancePage,
-  },
-  {
-    path: ':id/add',
-    loadChildren: () => import('../edit-allowance/edit-allowance.module').then(m => m.EditAllowancePageModule)
-  },
-  {
-    path: ':id/edit/:goalId',
-    loadChildren: () => import('../edit-allowance/edit-allowance.module').then(m => m.EditAllowancePageModule)
-  },
-  {
-    path: ':id/increase/:goalId',
-    loadChildren: () => import('../add-allowance/add-allowance.module').then(m => m.AddAllowancePageModule)
-  },
-  {
-    path: ':id/spend/:goalId',
-    loadChildren: () => import('../add-allowance/spend-allowance.module').then(m => m.SpendAllowancePageModule)
   }
 ];
 

frontend/allowance-planner-v2/src/app/pages/allowance/allowance.page.html

@@ -1,11 +1,8 @@
 <ion-header [translucent]="true" class="ion-no-border">
   <ion-toolbar>
-    <div class="toolbar">
+    <ion-title>
-      <ion-title>
+      Allowance
-        Allowance
+    </ion-title>
-      </ion-title>
-      <button class="top-add-button" (click)="createAllowance()">Add Goal</button>
-    </div>
   </ion-toolbar>
 </ion-header>
 
@@ -37,15 +34,15 @@
       <div class="main" *ngIf="goal.id === 0; else other_goal">
         <div class="title">
           <div class="name">Main Allowance</div>
-          <div class="icon" (click)="updateAllowance(goal.id)">
+          <div class="icon">
             <mat-icon>settings</mat-icon>
           </div>
         </div>
         <div class="progress">{{ goal.progress }} SP</div>
         <div class="buttons">
-          <button class="add-button" (click)="addAllowance(goal.id)">Add</button>
+          <button class="add-button">Add</button>
           <!-- <button class="move-button">Move</button> -->
-          <button class="spend-button" (click)="spendAllowance(goal.id)">Spend</button>
+          <button class="spend-button">Spend</button>
         </div>
       </div>
       <ng-template #other_goal>
@@ -53,15 +50,15 @@
           <div>
             <div class="title">
               <div class="name">{{ goal.name }}</div>
-              <div class="icon" (click)="updateAllowance(goal.id)">
+              <div class="icon">
                 <mat-icon>settings</mat-icon>
               </div>
             </div>
             <div class="progress">{{ goal.progress }} / {{ goal.target }} SP</div>
             <div class="buttons">
-              <button class="add-button" (click)="addAllowance(goal.id)">Add</button>
+              <button class="add-button">Add</button>
               <!-- <button class="move-button">Move</button> -->
-              <button class="spend-button" [disabled]="!canFinishGoal(goal)" (click)="completeGoal(goal.id)">Finish goal</button>
+              <button class="spend-button" [disabled]="!canFinishGoal(goal)">Finish goal</button>
             </div>
           </div>
           <div class="color" [style.--background]="hexToRgb(goal.colour)" [style.width.%]="getPercentage(goal)"></div>

frontend/allowance-planner-v2/src/app/pages/allowance/allowance.page.scss

@@ -58,10 +58,11 @@ button {
     padding-inline: 30px;
     border-radius: 10px;
     color: white;
+    font-size: 16px;
 }
 
 button:disabled,
-button[disabled] {
+button[disabled]{
     opacity: 0.5;
 }
 
@@ -126,14 +127,4 @@ button[disabled] {
     background-color: var(--legend-color);
     border-radius: 20px;
     margin-right: 2px;
-}
-
-.toolbar {
-    display: flex;
-}
-
-.top-add-button {
-    background-color: var(--ion-color-primary);
-    margin-right: 15px;
-    padding-inline: 15px;
 }

frontend/allowance-planner-v2/src/app/pages/allowance/allowance.page.ts

@@ -1,5 +1,5 @@
 import { Component } from '@angular/core';
-import { ActivatedRoute, Router } from '@angular/router';
+import { ActivatedRoute } from '@angular/router';
 import { BehaviorSubject } from 'rxjs';
 import { Allowance } from 'src/app/models/allowance';
 import { AllowanceService } from 'src/app/services/allowance.service';
@@ -14,11 +14,23 @@ import { ViewWillEnter } from '@ionic/angular';
 })
 export class AllowancePage implements ViewWillEnter {
   private id: number;
+  // Move to add/edit page later
+  private possibleColors: Array<string> = [
+    '#6199D9',
+    '#D98B61',
+    '#DBC307',
+    '#13DEB5',
+    '#7DCB7D',
+    '#CF1DBD',
+    '#F53311',
+    '#2F00FF',
+    '#098B0D',
+    '#1BC2E8'
+  ];
   public allowance$: BehaviorSubject<Array<Allowance>> = new BehaviorSubject<Array<Allowance>>([]);
 
   constructor(
     private route: ActivatedRoute,
-    private router: Router,
     private allowanceService: AllowanceService
   ) {
     this.id = this.route.snapshot.params['id'];
@@ -34,9 +46,10 @@ export class AllowancePage implements ViewWillEnter {
       this.allowanceService.getAllowanceList(this.id).subscribe(allowance => {
         allowance[0].colour = '#9C4BE4';
         allowance[0].name = 'Main Allowance';
+        console.log('Allowance list: ', allowance);
         this.allowance$.next(allowance);
       })
-    }, 100);
+    }, 10);
   }
 
   canFinishGoal(allowance: Allowance): boolean {
@@ -57,30 +70,6 @@ export class AllowancePage implements ViewWillEnter {
     for (let allowance of allowanceList) {
       allowanceTotal += allowance.progress;
     }
-    if (allowanceTotal === 0) {
-      return 0;
-    }
     return goal.progress / allowanceTotal * 100;
   }
-
-  createAllowance() {
-    this.router.navigate(['add'], { relativeTo: this.route });
-  }
-
-  updateAllowance(id: number) {
-    this.router.navigate(['edit', id], { relativeTo: this.route });
-  }
-
-  completeGoal(goalId: number) {
-    this.allowanceService.completeGoal(goalId, this.id);
-    this.getAllowance();
-  }
-
-  addAllowance(id: number) {
-    this.router.navigate(['increase', id], { relativeTo: this.route });
-  }
-
-  spendAllowance(id: number) {
-    this.router.navigate(['spend', id], { relativeTo: this.route });
-  }
 }

frontend/allowance-planner-v2/src/app/pages/edit-allowance/edit-allowance-routing.module.ts

@@ -1,17 +0,0 @@
-import { NgModule } from '@angular/core';
-import { Routes, RouterModule } from '@angular/router';
-
-import { EditAllowancePage } from './edit-allowance.page';
-
-const routes: Routes = [
-  {
-    path: '',
-    component: EditAllowancePage
-  }
-];
-
-@NgModule({
-  imports: [RouterModule.forChild(routes)],
-  exports: [RouterModule],
-})
-export class EditAllowancePageRoutingModule {}

frontend/allowance-planner-v2/src/app/pages/edit-allowance/edit-allowance.module.ts

@@ -1,25 +0,0 @@
-import { NgModule } from '@angular/core';
-import { CommonModule } from '@angular/common';
-import { FormsModule, ReactiveFormsModule } from '@angular/forms';
-
-import { IonicModule } from '@ionic/angular';
-
-import { EditAllowancePageRoutingModule } from './edit-allowance-routing.module';
-
-import { EditAllowancePage } from './edit-allowance.page';
-import { MatIconModule } from '@angular/material/icon';
-import { MatSelectModule } from '@angular/material/select';
-
-@NgModule({
-  imports: [
-    CommonModule,
-    FormsModule,
-    IonicModule,
-    EditAllowancePageRoutingModule,
-    ReactiveFormsModule,
-    MatIconModule,
-    MatSelectModule
-  ],
-  declarations: [EditAllowancePage]
-})
-export class EditAllowancePageModule {}

frontend/allowance-planner-v2/src/app/pages/edit-allowance/edit-allowance.page.html

@@ -1,47 +0,0 @@
-<ion-header [translucent]="true">
-  <ion-toolbar>
-    <div class="toolbar">
-      <div class="icon" (click)="navigateBack()">
-        <mat-icon>arrow_back</mat-icon>
-      </div>
-      <ion-title *ngIf="isAddMode">Create Goal</ion-title>
-      <ion-title *ngIf="!isAddMode && goalId != 0">Edit Goal</ion-title>
-      <ion-title *ngIf="!isAddMode && goalId == 0">Edit Allowance</ion-title>
-    </div>
-  </ion-toolbar>
-</ion-header>
-
-<ion-content [fullscreen]="true">
-  <form [formGroup]="form">
-    <div class="item" *ngIf="isAddMode || goalId != 0">
-      <label>Goal Name</label>
-      <input id="name" type="text" formControlName="name"/>
-    </div>
-
-    <div class="item" *ngIf="isAddMode || goalId != 0">
-      <label>Target</label>
-      <input id="target" type="number" placeholder="0.00" name="price" min="0" value="0" step="0.01" formControlName="target"/>
-    </div>
-
-    <label>Weight</label>
-    <input id="weight" type="number" placeholder="0.00" name="price" min="0" value="0" step="0.01" formControlName="weight"/>
-
-    <div class="item" *ngIf="isAddMode || goalId != 0">
-      <label>Colour</label>
-      <mat-select [(value)]="selectedColor" formControlName="color" [style.--color]="selectedColor">
-        <mat-option *ngFor="let color of possibleColors" [value]="color" [style.--background]="color">{{color}}</mat-option>
-      </mat-select>
-    </div>
-
-    <button type="button" [disabled]="!form.valid" (click)="submit()">
-      <span *ngIf="isAddMode">Add Goal</span>
-      <span *ngIf="!isAddMode && goalId != 0">Update Goal</span>
-      <span *ngIf="!isAddMode && goalId == 0">Update Allowance</span>
-    </button>
-    <button
-      *ngIf="!isAddMode && goalId !=0"
-      class="remove-button"
-      (click)="deleteAllowance()"
-    >Delete Goal</button>
-  </form>
-</ion-content>

frontend/allowance-planner-v2/src/app/pages/edit-allowance/edit-allowance.page.scss

@@ -1,64 +0,0 @@
-.toolbar {
-    display: flex;
-    align-items: center;
-}
-
-.remove-button {
-    margin-top: 10px;
-    background-color: var(--negative-amount-color);
-}
-
-form {
-    height: 100%;
-}
-
-form,
-.item {
-    display: flex;
-    flex-direction: column;
-    align-items: center;
-}
-
-label {
-    color: var(--ion-color-primary);
-    margin-top: 25px;
-    margin-bottom: 10px;
-}
-
-input,
-mat-select {
-    --color: black;
-    color: var(--color);
-    border: 1px solid var(--ion-color-primary);
-    border-radius: 5px;
-    width: 250px;
-    height: 40px;
-    padding-inline: 10px;
-    display: flex;
-    align-items: center;
-    font-family: (--ion-font-family);
-}
-
-mat-option {
-    --background: white;
-    color: var(--background);
-    font-family: (--ion-font-family);
-}
-
-button {
-    background-color: var(--ion-color-primary);
-    border-radius: 5px;
-    color: white;
-    padding: 10px;
-    width: 250px;
-    margin-top: 100px;
-}
-
-button:disabled,
-button[disabled]{
-    opacity: 0.5;
-}
-
-.icon {
-    margin-left: 5px;
-}

frontend/allowance-planner-v2/src/app/pages/edit-allowance/edit-allowance.page.spec.ts

@@ -1,17 +0,0 @@
-import { ComponentFixture, TestBed } from '@angular/core/testing';
-import { EditAllowancePage } from './edit-allowance.page';
-
-describe('EditAllowancePage', () => {
-  let component: EditAllowancePage;
-  let fixture: ComponentFixture<EditAllowancePage>;
-
-  beforeEach(() => {
-    fixture = TestBed.createComponent(EditAllowancePage);
-    component = fixture.componentInstance;
-    fixture.detectChanges();
-  });
-
-  it('should create', () => {
-    expect(component).toBeTruthy();
-  });
-});

frontend/allowance-planner-v2/src/app/pages/edit-allowance/edit-allowance.page.ts

@@ -1,109 +0,0 @@
-import { Location } from '@angular/common';
-import { Component, OnInit } from '@angular/core';
-import { FormBuilder, FormGroup, Validators } from '@angular/forms';
-import { ActivatedRoute, Router } from '@angular/router';
-import { AllowanceService } from 'src/app/services/allowance.service';
-
-@Component({
-  selector: 'app-edit-allowance',
-  templateUrl: './edit-allowance.page.html',
-  styleUrls: ['./edit-allowance.page.scss'],
-  standalone: false
-})
-export class EditAllowancePage implements OnInit {
-  public form: FormGroup;
-  public goalId: number;
-  public userId: number;
-  public isAddMode: boolean;
-  public selectedColor: string = '';
-  public possibleColors: Array<string> = [
-    '#6199D9',
-    '#D98B61',
-    '#DBC307',
-    '#13DEB5',
-    '#7DCB7D',
-    '#CF1DBD',
-    '#F53311',
-    '#2F00FF',
-    '#098B0D',
-    '#1BC2E8'
-  ];
-
-  constructor(
-    private route: ActivatedRoute,
-    private formBuilder: FormBuilder,
-    private allowanceService: AllowanceService,
-    private router: Router,
-    private location: Location
-  ) {
-    this.userId = this.route.snapshot.params['id'];
-    this.goalId = this.route.snapshot.params['goalId'];
-    this.isAddMode = !this.goalId;
-
-    this.allowanceService.getAllowanceList(this.userId).subscribe((list) => {
-      for (let allowance of list) {
-        this.possibleColors = this.possibleColors.filter(color => color !== allowance.colour);
-        if (!this.isAddMode && +this.goalId === allowance.id) {
-          this.possibleColors.unshift(allowance.colour);
-        }
-      }
-    });
-
-    this.form = this.formBuilder.group({
-      name: ['', Validators.required],
-      target: ['', Validators.required],
-      weight: ['', Validators.required],
-      color: ['', Validators.required]
-    });
-  }
-
-  ngOnInit() {
-    if (!this.isAddMode) {
-      this.allowanceService.getAllowanceById(this.goalId, this.userId).subscribe((allowance) => {
-        if (+this.goalId === 0) {
-          this.form.setValue({
-            name: 'Main Allowance',
-            target: 0,
-            weight: allowance.weight,
-            color: '#9C4BE4'
-          });
-        } else {
-          this.form.setValue({
-            name: allowance.name,
-            target: allowance.target,
-            weight: allowance.weight,
-            color: allowance.colour
-          });
-          this.selectedColor = this.form.value.color;
-        }
-      });
-    }
-  }
-
-  submit() {
-    const formValue = this.form.value;
-    const allowance = {
-      name: formValue.name,
-      target: formValue.target,
-      weight: formValue.weight,
-      colour: formValue.color,
-    };
-
-    if (this.isAddMode) {
-      this.allowanceService.createAllowance(allowance, this.userId);
-    } else {
-      this.allowanceService.updateAllowance(allowance, this.goalId, this.userId);
-    }
-
-    this.router.navigate(['/tabs/allowance', this.userId]);
-  }
-
-  deleteAllowance() {
-    this.allowanceService.deleteAllowance(this.goalId, this.userId);
-    this.router.navigate(['/tabs/allowance', this.userId]);
-  }
-
-  navigateBack() {
-    this.location.back();
-  }
-}

frontend/allowance-planner-v2/src/app/pages/edit-task/edit-task.module.ts

@@ -7,8 +7,6 @@ import { IonicModule } from '@ionic/angular';
 import { EditTaskPageRoutingModule } from './edit-task-routing.module';
 
 import { EditTaskPage } from './edit-task.page';
-import { MatIconModule } from '@angular/material/icon';
-import { MatSelectModule } from '@angular/material/select';
 
 @NgModule({
   imports: [
@@ -16,9 +14,7 @@ import { MatSelectModule } from '@angular/material/select';
     FormsModule,
     IonicModule,
     EditTaskPageRoutingModule,
-    ReactiveFormsModule,
+    ReactiveFormsModule
-    MatIconModule,
-    MatSelectModule
   ],
   declarations: [EditTaskPage]
 })

frontend/allowance-planner-v2/src/app/pages/edit-task/edit-task.page.html

@@ -1,11 +1,13 @@
 <ion-header [translucent]="true">
   <ion-toolbar>
     <div class="toolbar">
-      <div class="icon" (click)="navigateBack()">
-        <mat-icon>arrow_back</mat-icon>
-      </div>
       <ion-title *ngIf="isAddMode">Create Task</ion-title>
       <ion-title *ngIf="!isAddMode">Edit Task</ion-title>
+      <button
+        *ngIf="!isAddMode"
+        class="remove-button"
+        (click)="deleteTask()"
+      >Delete task</button>
     </div>
   </ion-toolbar>
 </ion-header>
@@ -16,21 +18,16 @@
     <input id="name" type="text" formControlName="name"/>
 
     <label>Reward</label>
-    <input id="reward" type="number" placeholder="0.00" name="price" min="0" value="0" step="0.01" formControlName="reward"/>
+    <input id="name" type="number" placeholder="0.00" name="price" min="0" value="0" step="0.01" formControlName="reward"/>
 
     <label>Assigned</label>
-    <mat-select formControlName="assigned">
+    <select formControlName="assigned">
-      <mat-option *ngFor="let user of users" [value]="user.id">{{ user.name }}</mat-option>
+      <option *ngFor="let user of users" [value]="user.id">{{ user.name }}</option>
-    </mat-select>
+    </select>
 
     <button type="button" [disabled]="!form.valid" (click)="submit()">
       <span *ngIf="isAddMode">Add Task</span>
       <span *ngIf="!isAddMode">Update Task</span>
     </button>
-    <button
-      *ngIf="!isAddMode"
-      class="remove-button"
-      (click)="deleteTask()"
-    >Delete task</button>
   </form>
 </ion-content>

frontend/allowance-planner-v2/src/app/pages/edit-task/edit-task.page.scss

@@ -1,11 +1,12 @@
 .toolbar {
     display: flex;
-    align-items: center;
 }
 
 .remove-button {
-    margin-top: 10px;
+    background-color: var(--ion-color-primary);
-    background-color: var(--negative-amount-color);
+    margin-right: 15px;
+    width: 85px;
+    margin-bottom: 0;
 }
 
 form {
@@ -22,15 +23,10 @@ label {
 }
 
 input,
-mat-select {
+select {
     border: 1px solid var(--ion-color-primary);
     border-radius: 5px;
     width: 250px;
-    height: 40px;
-    padding-inline: 10px;
-    display: flex;
-    align-items: center;
-    font-family: (--ion-font-family);
 }
 
 button {
@@ -39,14 +35,11 @@ button {
     color: white;
     padding: 10px;
     width: 250px;
-    margin-top: 100px;
+    margin-top: auto;
+    margin-bottom: 50px;
 }
 
 button:disabled,
 button[disabled]{
     opacity: 0.5;
-}
-
-.icon {
-    margin-left: 5px;
 }

frontend/allowance-planner-v2/src/app/pages/edit-task/edit-task.page.ts

@@ -23,8 +23,7 @@ export class EditTaskPage implements OnInit {
     private formBuilder: FormBuilder,
     private taskService: TaskService,
     private userService: UserService,
-    private router: Router,
+    private router: Router
-    private location: Location
   ) {
     this.id = this.route.snapshot.params['id'];
     this.isAddMode = !this.id;
@@ -78,8 +77,4 @@ export class EditTaskPage implements OnInit {
     this.taskService.deleteTask(this.id);
     this.router.navigate(['/tabs/tasks']);
   }
-
-  navigateBack() {
-    this.location.back();
-  }
 }

frontend/allowance-planner-v2/src/app/pages/history/history.module.ts

@@ -5,8 +5,6 @@ import { FormsModule } from '@angular/forms';
 import { HistoryPage } from './history.page';
 
 import { HistoryPageRoutingModule } from './history-routing.module';
-import { provideHttpClient } from '@angular/common/http';
-import { HistoryService } from 'src/app/services/history.service';
 
 @NgModule({
   imports: [
@@ -15,10 +13,6 @@ import { HistoryService } from 'src/app/services/history.service';
     FormsModule,
     HistoryPageRoutingModule
   ],
-  declarations: [HistoryPage],
+  declarations: [HistoryPage]
-  providers: [
-    provideHttpClient(),
-    HistoryService
-  ]
 })
 export class HistoryPageModule {}

frontend/allowance-planner-v2/src/app/pages/history/history.page.html

@@ -7,14 +7,5 @@
 </ion-header>
 
 <ion-content>
-  <div class="item" *ngFor="let history of history$ | async">
+
-    <div class="left">
-      <div class="date">{{ history.timestamp | date: 'yyyy-MM-dd' }}</div>
-      <div class="description">{{ history.description }}</div>
-    </div>
-    <div
-      class="amount"
-      [ngClass]="{ 'negative': history.allowance < 0 }"
-    >{{ history.allowance }} SP</div>
-  </div>
 </ion-content>

frontend/allowance-planner-v2/src/app/pages/history/history.page.scss

@@ -1,30 +0,0 @@
-.item {
-    display: flex;
-    flex-direction: row;
-    align-items: center;
-    border-bottom: 1px solid var(--line-color);
-    padding: 8px;
-}
-
-.left {
-    width: 70%;
-    font-size: 18px;
-}
-
-.date {
-    color: var(--line-color);
-}
-
-.description {
-    color: var(--font-color);
-}
-
-.amount {
-    margin-left: auto;
-    font-size: 22px;
-    color: var(--positive-amount-color);
-}
-
-.negative {
-    color: var(--negative-amount-color);
-}

frontend/allowance-planner-v2/src/app/pages/history/history.page.ts

@@ -1,9 +1,4 @@
 import { Component } from '@angular/core';
-import { ActivatedRoute } from '@angular/router';
-import { ViewWillEnter } from '@ionic/angular';
-import { BehaviorSubject } from 'rxjs';
-import { History } from 'src/app/models/history';
-import { HistoryService } from 'src/app/services/history.service';
 
 @Component({
   selector: 'app-history',
@@ -11,28 +6,8 @@ import { HistoryService } from 'src/app/services/history.service';
   styleUrls: ['history.page.scss'],
   standalone: false,
 })
-export class HistoryPage implements ViewWillEnter {
+export class HistoryPage {
-  userId: number;
-  public history$: BehaviorSubject<Array<History>> = new BehaviorSubject<Array<History>>([]);
-  
 
-  constructor(
+  constructor() {}
-    private route: ActivatedRoute,
-    private historyService: HistoryService
-  ) {
-    this.userId = this.route.snapshot.params['id'];
-    this.getHistory();
-  }
 
-  ionViewWillEnter(): void {
-    this.getHistory();
-  }
-
-  getHistory() {
-    setTimeout(() => {
-      this.historyService.getHistoryList(this.userId).subscribe(history => {
-        this.history$.next(history);
-      })
-    }, 20);
-  }
 }

frontend/allowance-planner-v2/src/app/pages/tabs/tabs-routing.module.ts

@@ -8,7 +8,7 @@ const routes: Routes = [
     component: TabsPage,
     children: [
       {
-        path: 'history/:id',
+        path: 'history',
         loadChildren: () => import('../history/history.module').then(m => m.HistoryPageModule)
       },
       {

frontend/allowance-planner-v2/src/app/pages/tabs/tabs.page.html

@@ -1,6 +1,6 @@
 <ion-tabs>
   <ion-tab-bar slot="bottom">
-    <ion-tab-button [tab]="historyTab" [href]="historyNav">
+    <ion-tab-button tab="history" href="/tabs/history">
       <mat-icon>history</mat-icon>
     </ion-tab-button>
     <ion-tab-button tab="allowance" href="/tabs/allowance">

frontend/allowance-planner-v2/src/app/pages/tabs/tabs.page.ts

@@ -1,5 +1,4 @@
 import { Component } from '@angular/core';
-import { StorageService } from 'src/app/services/storage.service';
 
 @Component({
   selector: 'app-tabs',
@@ -8,16 +7,6 @@ import { StorageService } from 'src/app/services/storage.service';
   standalone: false,
 })
 export class TabsPage {
-  historyNav = '';
+  constructor() {}
-  historyTab = '';
-
-  constructor(private storageService: StorageService) {
-    this.storageService.getCurrentUserId().then((userId) => {
-        if (userId !== undefined && userId !== null) {
-          this.historyNav = `/tabs/history/${userId}`;
-          this.historyTab = `history/${userId}`;
-        }
-      });
-  }
 
 }

frontend/allowance-planner-v2/src/app/pages/tasks/tasks.page.html

@@ -11,19 +11,14 @@
 
 <ion-content>
   <div class="content">
-    <!-- <div class="icon">
+    <div class="icon">
       <mat-icon>filter_alt</mat-icon>
-    </div> -->
+    </div>
     <div class="list">
       <div class="task" *ngFor="let task of tasks$ | async">
         <button (click)="completeTask(task.id)">Done</button>
         <div (click)="updateTask(task.id)" class="item">
-          <div class="text">
+          <div class="name">{{ task.name }}</div>
-            <div class="name">
-              {{ task.name }}
-              <span class="assigned">{{ usernames[task.assigned ? task.assigned : 0] }}</span>
-            </div>
-          </div>
           <div
             class="reward"
             [ngClass]="{ 'negative': task.reward < 0 }"

frontend/allowance-planner-v2/src/app/pages/tasks/tasks.page.scss

@@ -31,8 +31,6 @@ mat-icon {
     align-items: center;
     border-bottom: 1px solid var(--line-color);
     padding: 5px;
-    padding-block: 10px;
-    font-size: 18px;
 }
 
 .item {
@@ -43,6 +41,7 @@ mat-icon {
 }
 
 .name {
+    margin-left: 10px;
     color: var(--font-color);
 }
 
@@ -50,7 +49,6 @@ mat-icon {
     margin-left: auto;
     margin-right: 15px;
     color: var(--positive-amount-color);
-    font-size: 22px;
 }
 
 .negative {
@@ -58,28 +56,15 @@ mat-icon {
 }
 
 button {
-    height: 45px;
+    width: 57px;
+    height: 30px;
     border-radius: 10px;
     color: white;
     background: var(--confirm-button-color);
-    padding-inline: 15px;
 }
 
 .add-button {
     background-color: var(--ion-color-primary);
     margin-right: 15px;
-    height: 30px;
+    width: 75px;
-}
-
-.assigned {
-    color: var(--line-color);
-    margin-left: 3px;
-    font-size: 15px;
-}
-
-.text {
-    display: flex;
-    align-items: center;
-    width: 60%;
-    margin-left: 10px;
 }

frontend/allowance-planner-v2/src/app/pages/tasks/tasks.page.ts

@@ -14,7 +14,6 @@ import { ViewWillEnter } from '@ionic/angular';
 })
 export class TasksPage implements ViewWillEnter {
   public tasks$: BehaviorSubject<Array<Task>> = new BehaviorSubject<Array<Task>>([]);
-  public usernames = ['', 'See', 'Huffle']
 
   constructor(
     private taskService: TaskService,
@@ -33,7 +32,7 @@ export class TasksPage implements ViewWillEnter {
       this.taskService.getTaskList().subscribe(tasks => {
         this.tasks$.next(tasks);
       });
-    }, 100);
+    }, 10);
   }
 
   createTask() {

frontend/allowance-planner-v2/src/app/services/allowance.service.ts

@@ -7,35 +7,11 @@ import { Allowance } from '../models/allowance';
     providedIn: 'root'
 })
 export class AllowanceService {
-    private url = 'https://allowanceplanner.seeseepuff.be/api';
+    private url = 'http://localhost:8080/api';
 
     constructor(private http: HttpClient) {}
 
     getAllowanceList(userId: number): Observable<Array<Allowance>> {
         return this.http.get<Allowance[]>(`${this.url}/user/${userId}/allowance`);
     }
-
-    getAllowanceById(allowanceId: number, userId: number): Observable<Allowance> {
-        return this.http.get<Allowance>(`${this.url}/user/${userId}/allowance/${allowanceId}`);
-    }
-
-    createAllowance(allowance: Partial<Allowance>, userId: number) {
-        this.http.post(`${this.url}/user/${userId}/allowance`, allowance).subscribe();
-    }
-
-    updateAllowance(allowance: Partial<Allowance>, allowanceId: number, userId: number) {
-        this.http.put(`${this.url}/user/${userId}/allowance/${allowanceId}`, allowance).subscribe();
-    }
-
-    deleteAllowance(allowanceId: number, userId: number) {
-        this.http.delete(`${this.url}/user/${userId}/allowance/${allowanceId}`).subscribe();
-    }
-
-    completeGoal(goalId: number, userId: number) {
-        this.http.post(`${this.url}/user/${userId}/allowance/${goalId}/complete`, {}).subscribe();
-    }
-
-    addOrSpendAllowance(goalId: number, userId: number, amount: number, description: string) {
-        this.http.post(`${this.url}/user/${userId}/allowance/${goalId}/add`, { amount, description }).subscribe();
-    }
 }

frontend/allowance-planner-v2/src/app/services/history.service.ts

@@ -1,17 +0,0 @@
-import { HttpClient } from '@angular/common/http';
-import { Injectable } from '@angular/core';
-import { Observable } from 'rxjs';
-import { History } from '../models/history';
-
-@Injectable({
-    providedIn: 'root'
-})
-export class HistoryService {
-    private url = 'https://allowanceplanner.seeseepuff.be/api';
-
-    constructor(private http: HttpClient) {}
-
-    getHistoryList(userId: number): Observable<Array<History>> {
-        return this.http.get<History[]>(`${this.url}/user/${userId}/history`);
-    }
-}

frontend/allowance-planner-v2/src/app/services/task.service.ts

@@ -7,7 +7,7 @@ import { Task } from '../models/task';
     providedIn: 'root'
 })
 export class TaskService {
-    private url = 'https://allowanceplanner.seeseepuff.be/api';
+    private url = 'http://localhost:8080/api';
 
     constructor(private http: HttpClient) {}
 

frontend/allowance-planner-v2/src/app/services/user.service.ts

@@ -7,7 +7,7 @@ import { User } from '../models/user';
     providedIn: 'root',
 })
 export class UserService {
-    private url = 'https://allowanceplanner.seeseepuff.be/api';
+    private url = 'http://localhost:8080/api';
     constructor(private http: HttpClient) {}
 
     getUserList(): Observable<Array<User>> {

frontend/allowance-planner-v2/src/global.scss

@@ -38,34 +38,8 @@
 
 ion-title {
     color: var(--ion-color-primary);
-    font-size: 24px;
 }
 
 ion-header {
     border-bottom: 1px solid var(--line-color);
-}
-
-button {
-    font-size: 16px;
-}
-
-ion-header.md {
-    ion-toolbar:first-child {
-        --padding-top: 30px;
-        --padding-bottom: 15px;
-    }
-}
-
-label {
-    font-size: 18px;
-}
-
-ion-alert .alert-wrapper.sc-ion-alert-md {
-    background-color: var(--ion-background-color) !important;
-    --background: unset !important;
-    box-shadow: unset;
-}
-
-ion-alert .alert-tappable.sc-ion-alert-md {
-    background-color: var(--test-color);
 }