chore: make event store accept IUser (#3076)

## About the changes Currently, we need to remember of using the email or else the username of a user when storing into EventStore, because we don't have [strictNullChecks](https://www.typescriptlang.org/tsconfig#strictNullChecks), it's error-prone. Fix for a production issue: #3072 This reuses an existing function that also deals with undefined
Unleash · Feb 21, 2023 · dc0fe39 · dc0fe39
1 parent 03c3499
commit dc0fe39
Show file tree

Hide file tree

Showing 3 changed files with 134 additions and 34 deletions.
diff --git a/src/lib/services/feature-toggle-service.ts b/src/lib/services/feature-toggle-service.ts
@@ -1350,7 +1350,7 @@ class FeatureToggleService {
                 featureName,
                 environment,
                 project: projectId,
-                createdBy: user.email || user.username,
+                createdBy: user,
                 oldVariants: theOldVariants,
                 newVariants: fixedVariants,
             }),
@@ -1436,7 +1436,7 @@ class FeatureToggleService {
                         featureName,
                         environment,
                         project: projectId,
-                        createdBy: user.email || user.username,
+                        createdBy: user,
                         oldVariants: oldVariants[environment],
                         newVariants: fixedVariants,
                     }),