Merge pull request #1379 from openmeterio/notification-api

fix: missing payload in notification event
openmeterio · Aug 15, 2024 · d618574 · d618574
2 parents c002ba4 + 8f35441
commit d618574
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/internal/notification/event.go b/internal/notification/event.go
@@ -205,7 +205,13 @@ func (p EventPayload) AsNotificationEventBalanceThresholdPayload(eventId string,
 			Subject     api.Subject                               `json:"subject"`
 			Threshold   api.NotificationRuleBalanceThresholdValue `json:"threshold"`
 			Value       api.EntitlementValue                      `json:"value"`
-		}{},
+		}{
+			Entitlement: p.BalanceThreshold.Entitlement,
+			Feature:     p.BalanceThreshold.Feature,
+			Subject:     p.BalanceThreshold.Subject,
+			Threshold:   p.BalanceThreshold.Threshold,
+			Value:       p.BalanceThreshold.Value,
+		},
 		Id:        eventId,
 		Timestamp: ts,
 		Type:      api.EntitlementsBalanceThreshold,
@@ -471,7 +477,7 @@ func PayloadToMapInterface(t any) (map[string]interface{}, error) {
 	}
 
 	var m map[string]interface{}
-	if err := json.Unmarshal(b, &m); err != nil {
+	if err = json.Unmarshal(b, &m); err != nil {
 		return nil, err
 	}