fix: more todos

Author: turip

openmeter/entitlement/balanceworker/entitlementhandler.go

@@ -163,6 +163,7 @@ func (w *Worker) processEntitlementEntity(ctx context.Context, entitlementEntity
 
 	var err error
 	var snapshot marshaler.Event
+	// TODO: This must not be in scope for high watermark cache or we are adding negative estimations!!!
 	if opts.useEstimator && entitlementEntity.EntitlementType == entitlement.EntitlementTypeMetered {
 		snapshot, err = w.createSnapshotEventEstimator(ctx, entitlementEntity, calculatedAt, opts)
 		if err != nil {

openmeter/entitlement/balanceworker/entitlementthreshold.go

@@ -21,6 +21,8 @@ func (w *Worker) hitsWatchedThresholds(ctx context.Context, ent entitlement.Enti
 			return false, fmt.Errorf("failed to get next active thresholds: %w", err)
 		}
 
+		// TODO: Let's check if overage / usage is correctly handled in the code (e.g. > 100% notifications etc)
+
 		if nextThreshold != nil {
 			if entitlementEnt.ApproxUsage.GreaterThanOrEqual(estimator.NewInfDecimalFromDecimal(*nextThreshold)) {
 				return true, nil