docs(comments): add some comments

api/handlers/orders.go

@@ -5,12 +5,11 @@ import (
 	"net/http"
 	"os"
 	"strconv"
-	"time"
-
 	"terminalShop/pkg/bring"
 	"terminalShop/pkg/notify"
 	"terminalShop/pkg/shipping"
 	"terminalShop/pkg/shippo"
+	"time"
 
 	"terminalShop/api/middleware"
 	"terminalShop/pkg/audit"
@@ -21,7 +20,6 @@ import (
 	"github.com/go-chi/chi/v5"
 	"github.com/stripe/stripe-go/v78"
 	"github.com/stripe/stripe-go/v78/refund"
-	"gorm.io/gorm"
 )
 
 type OrderHandler struct {
@@ -45,10 +43,7 @@ func (h *OrderHandler) GetOrders(w http.ResponseWriter, r *http.Request) {
 	userID := middleware.UserIDFromContext(r.Context())
 
 	var orders []models.Order
-	db.Where("user_id = ?", userID).
-		Preload("Items").
-		Preload("Events", func(tx *gorm.DB) *gorm.DB { return tx.Order("created_at ASC") }).
-		Order("created_at desc").Find(&orders)
+	db.Where("user_id = ?", userID).Preload("Items").Order("created_at desc").Find(&orders)
 
 	utils.RespondSuccess(w, http.StatusOK, map[string]interface{}{
 		"orders": orders,

api/handlers/reconcile.go

@@ -7,7 +7,6 @@ import (
 
 	"terminalShop/pkg/database"
 	"terminalShop/pkg/models"
-	"terminalShop/pkg/notify"
 
 	"github.com/stripe/stripe-go/v78"
 	"github.com/stripe/stripe-go/v78/paymentintent"
@@ -76,26 +75,3 @@ func ReconcileOrders(stripeKey string) {
 		}
 	}
 }
-
-// ReconcileUnshipped finds paid order orlder than 24h with no tracking
-// number and posts single Slack reminder. Pure read + notif
-func ReconcileUnshipped() {
-	db := database.GetDB()
-
-	var orders []models.Order
-	cutoff := time.Now().Add(-24 * time.Hour)
-	err := db.Where(
-		"status = ? AND (tracking_number = '' OR tracking_number IS NULL) AND created_at < ?",
-		models.OrderStatusPaid, cutoff,
-	).Order("created_at ASC").Find(&orders).Error
-	if err != nil {
-		reconcileLog().Error("unshipped query failed", "error", err)
-		return
-	}
-	if len(orders) == 0 {
-		return
-	}
-
-	reconcileLog().Info("unshipped orders found", "count", len(orders))
-	notify.SlackUnshippedReminder(orders)
-}

api/handlers/view.go

@@ -7,8 +7,6 @@ import (
 	"terminalShop/pkg/database"
 	"terminalShop/pkg/models"
 	"terminalShop/pkg/utils"
-
-	"gorm.io/gorm"
 )
 
 type ViewHandler struct{}
@@ -40,10 +38,7 @@ func (h *ViewHandler) GetViewInit(w http.ResponseWriter, r *http.Request) {
 	db.Where("user_id = ?", userID).Find(&cards)
 
 	var orders []models.Order
-	db.Where("user_id = ?", userID).
-		Preload("Items").
-		Preload("Events", func(tx *gorm.DB) *gorm.DB { return tx.Order("created_at ASC") }).
-		Order("created_at DESC").Find(&orders)
+	db.Where("user_id = ?", userID).Preload("Items").Order("created_at DESC").Find(&orders)
 	var cartItems []models.CartItem
 	if cart.ID != 0 {
 		db.Where("cart_id = ? AND quantity > 0", cart.ID).Preload("Coffee").Find(&cartItems)

api/main.go

@@ -113,21 +113,6 @@ func main() {
 		}
 	}()
 
-	go func() {
-		time.Sleep(30 * time.Second)
-		handlers.ReconcileUnshipped()
-		ticker := time.NewTicker(24 * time.Hour)
-		defer ticker.Stop()
-		for {
-			select {
-			case <-ticker.C:
-				handlers.ReconcileUnshipped()
-			case <-ctx.Done():
-				return
-			}
-		}
-	}()
-
 	log.Println("API server started. Press Ctrl+C to shutdown.")
 
 	// Block until signal received

pkg/models/order.go

@@ -71,10 +71,9 @@ type Order struct {
 	DeletedAt gorm.DeletedAt `gorm:"index" json:"-"` // GORM soft-delete sentinel
 
 	// Associations
-	User   User         `gorm:"foreignKey:UserID" json:"-"`
-	Card   Card         `gorm:"foreignKey:CardID" json:"-"`
-	Items  []OrderItem  `gorm:"foreignKey:OrderID" json:"items"`
-	Events []OrderEvent `gorm:"foreignKey:OrderID" json:"events,omitempty"`
+	User  User        `gorm:"foreignKey:UserID" json:"-"`
+	Card  Card        `gorm:"foreignKey:CardID" json:"-"`
+	Items []OrderItem `gorm:"foreignKey:OrderID" json:"items"`
 }
 
 // TableName specifies the table name for the Order model.

pkg/models/order_event.go

@@ -1,9 +1,6 @@
 package models
 
-import (
-	"encoding/json"
-	"time"
-)
+import "time"
 
 type OrderEvent struct {
 	ID        uint      `gorm:"primaryKey" json:"id"`
@@ -17,47 +14,3 @@ type OrderEvent struct {
 func (OrderEvent) TableName() string {
 	return "order_events"
 }
-
-func (e *OrderEvent) PayloadMap() map[string]any {
-	if e.Payload == "" {
-		return nil
-	}
-	var m map[string]any
-	if err := json.Unmarshal([]byte(e.Payload), &m); err != nil {
-		return nil
-	}
-	return m
-}
-
-func (e *OrderEvent) DisplayLabel() (string, bool) {
-	switch e.Type {
-	case "order_paid":
-		return "Paid", true
-	case "label_purchased":
-		return "Label created", true
-	case "order_delivered":
-		return "Delivered", true
-	case "order_refunded":
-		return "Refunded", true
-	case "order_failed":
-		return "Payment failed", true
-	case "tracking_updated":
-		status, _ := e.PayloadMap()["status"].(string)
-		return trackingLabel(status)
-	}
-	return "", false
-}
-
-func trackingLabel(status string) (string, bool) {
-	switch status {
-	case "transit":
-		return "In transit", true
-	case "failure":
-		return "Delivery issue", true
-	case "returned":
-		return "Returned to sender", true
-	case "pre_transit":
-		return "Label scanned by carrier", true
-	}
-	return "", false
-}

pkg/notify/slack.go

@@ -1,4 +1,3 @@
-// Package notify provides a simple interface to send notifications to Slack.
 package notify
 
 import (
@@ -15,7 +14,6 @@ import (
 	"terminalShop/pkg/models"
 )
 
-// slack client
 var slackClient = &http.Client{Timeout: 5 * time.Second}
 
 // dollars converts cents to dollars
@@ -264,44 +262,4 @@ func markStatusActionBlock(orderID uint) map[string]any {
 	}
 }
 
-func SlackUnshippedReminder(orders []models.Order) {
-	token := os.Getenv("SLACK_BOT_TOKEN")
-	channel := os.Getenv("SLACK_CHANNEL")
-	if token == "" || channel == "" || len(orders) == 0 {
-		return
-	}
-
-	workspace := os.Getenv("SLACK_WORKSPACE_SUBDOMAIN")
-
-	lines := []string{fmt.Sprintf(":hourglass_flowing_sand: %d order(s) awaiting label", len(orders))}
-	for _, order := range orders {
-		lines = append(lines, " "+unshippedOrderLine(order, channel, workspace))
-	}
-
-	_, _ = postSlackMessage(token, map[string]any{
-		"channel": channel,
-		"text":    strings.Join(lines, "\n"),
-	})
-}
-
-func unshippedOrderLine(order models.Order, channel, workspace string) string {
-	age := time.Since(order.CreatedAt).Round(time.Hour)
-	location := order.ShippingCountry
-	if order.ShippingCity != "" {
-		location = order.ShippingCity + ", " + order.ShippingCountry
-	}
-
-	label := fmt.Sprintf("Order #%d", order.ID)
-	if order.SlackThreadTS != nil && *order.SlackThreadTS != "" && workspace != "" {
-		ts := strings.ReplaceAll(*order.SlackThreadTS, ".", "")
-		permalink := fmt.Sprintf("https://%s.slack.com/archives/%s/p%s", workspace, channel, ts)
-		label = fmt.Sprintf("<%s|Order #%d>", permalink, order.ID)
-	}
-
-	tag := ""
-	if order.ShippingCountry == "NO" {
-		tag = " _(NO * Bring blocked)_"
-	}
-
-	return fmt.Sprintf("%s * %s old * %s * %s%s", label, age, dollars(order.Total), location, tag)
-}
+var _ = time.Second

pkg/tui/account_orders.go

@@ -201,24 +201,6 @@ func (m Model) buildOrderDetailView(order models.Order, _ int) string {
 		}
 	}
 
-	visible := make([]models.OrderEvent, 0, len(order.Events))
-	for _, event := range order.Events {
-		if _, ok := event.DisplayLabel(); ok {
-			visible = append(visible, event)
-		}
-	}
-	if len(visible) > 0 {
-		b.WriteString(labelStyle.Render("Timeline"))
-		b.WriteString("\n")
-		for _, event := range visible {
-			label, _ := event.DisplayLabel()
-			when := event.CreatedAt.Format("Jan 02 3:04 PM")
-			b.WriteString(valueStyle.Render(fmt.Sprintf("  %-20s %s", label, dimStyle.Render(when))))
-			b.WriteString("\n")
-		}
-		b.WriteString("\n")
-	}
-
 	// Shipping address
 	b.WriteString(labelStyle.Render("Ship To"))
 	b.WriteString("\n")