refactor(events): remove redundant PostHog enqueue and add debug logging for point grants

pan93412 · pan93412 · commit a48d523c0808 · 2025-10-06T21:55:11.000+08:00
diff --git a/internal/events/points.go b/internal/events/points.go
@@ -389,12 +389,6 @@ func (d *PointsGranter) GrantFirstPlacePoints(ctx context.Context, userID int, q
 		return false, err
 	}
 
-	if d.posthogClient != nil {
-		d.posthogClient.Enqueue(posthog.Capture{
-			DistinctId: strconv.Itoa(userID),
-		})
-	}
-
 	return true, nil
 }
 
@@ -424,6 +418,8 @@ func (d *PointsGranter) grantPoint(ctx context.Context, userID int, questionID i
 			properties.Set("questionID", strconv.Itoa(questionID))
 		}
 
+		slog.Debug("sending event to PostHog", "event_type", GRANT_POINT_EVENT, "user_id", userID)
+
 		d.posthogClient.Enqueue(posthog.Capture{
 			DistinctId: strconv.Itoa(userID),
 			Event:      GRANT_POINT_EVENT,

Original file line number	Diff line number	Diff line change
`@@ -389,12 +389,6 @@ func (d *PointsGranter) GrantFirstPlacePoints(ctx context.Context, userID int, q`
`389`	`389`	`return false, err`
`390`	`390`	`}`
`391`	`391`
`392`		`- if d.posthogClient != nil {`
`393`		`- d.posthogClient.Enqueue(posthog.Capture{`
`394`		`- DistinctId: strconv.Itoa(userID),`
`395`		`- })`
`396`		`- }`
`397`		`-`
`398`	`392`	`return true, nil`
`399`	`393`	`}`
`400`	`394`
`@@ -424,6 +418,8 @@ func (d *PointsGranter) grantPoint(ctx context.Context, userID int, questionID i`
`424`	`418`	`properties.Set("questionID", strconv.Itoa(questionID))`
`425`	`419`	`}`
`426`	`420`
	`421`	`+ slog.Debug("sending event to PostHog", "event_type", GRANT_POINT_EVENT, "user_id", userID)`
	`422`	`+`
`427`	`423`	`d.posthogClient.Enqueue(posthog.Capture{`
`428`	`424`	`DistinctId: strconv.Itoa(userID),`
`429`	`425`	`Event: GRANT_POINT_EVENT,`