From 139adf604e9dfd8ab0bfcb433cfa90d70fb38897 Mon Sep 17 00:00:00 2001 From: Austin Rhode Date: Thu, 20 Jun 2024 17:59:11 +0000 Subject: [PATCH] feat: added status to the return value of ConvertUsers and removed logic to ignore non-active users when creating a snapshot --- coderd/telemetry/telemetry.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/coderd/telemetry/telemetry.go b/coderd/telemetry/telemetry.go index 36292179da478..9b371d015ec18 100644 --- a/coderd/telemetry/telemetry.go +++ b/coderd/telemetry/telemetry.go @@ -353,9 +353,6 @@ func (r *remoteReporter) createSnapshot() (*Snapshot, error) { users := database.ConvertUserRows(userRows) var firstUser database.User for _, dbUser := range users { - if dbUser.Status != database.UserStatusActive { - continue - } if firstUser.CreatedAt.IsZero() { firstUser = dbUser } @@ -655,6 +652,7 @@ func ConvertUser(dbUser database.User) User { EmailHashed: emailHashed, RBACRoles: dbUser.RBACRoles, CreatedAt: dbUser.CreatedAt, + Status: dbUser.Status, } }