Merge pull request #1760 from zed-industries/invite-unknown-platform

Max Brunsfeld created 3 years ago

Include waitlist entries w/ unknown platform when summarizing and sending invites

Change summary

crates/collab/src/db.rs       | 7 +++++--
crates/collab/src/db_tests.rs | 2 ++
2 files changed, 7 insertions(+), 2 deletions(-)

Detailed changes

crates/collab/src/db.rs

@@ -428,7 +428,8 @@ impl Db for PostgresDb {
                 COUNT(*) as count,
                 COALESCE(SUM(CASE WHEN platform_linux THEN 1 ELSE 0 END), 0) as linux_count,
                 COALESCE(SUM(CASE WHEN platform_mac THEN 1 ELSE 0 END), 0) as mac_count,
-                COALESCE(SUM(CASE WHEN platform_windows THEN 1 ELSE 0 END), 0) as windows_count
+                COALESCE(SUM(CASE WHEN platform_windows THEN 1 ELSE 0 END), 0) as windows_count,
+                COALESCE(SUM(CASE WHEN platform_unknown THEN 1 ELSE 0 END), 0) as unknown_count
             FROM (
                 SELECT *
                 FROM signups
@@ -449,7 +450,7 @@ impl Db for PostgresDb {
             FROM signups
             WHERE
                 NOT email_confirmation_sent AND
-                platform_mac
+                (platform_mac OR platform_unknown)
             LIMIT $1
             ",
         )
@@ -1720,6 +1721,8 @@ pub struct WaitlistSummary {
     pub mac_count: i64,
     #[sqlx(default)]
     pub windows_count: i64,
+    #[sqlx(default)]
+    pub unknown_count: i64,
 }
 
 #[derive(FromRow, PartialEq, Debug, Serialize, Deserialize)]

crates/collab/src/db_tests.rs

@@ -1022,6 +1022,7 @@ async fn test_signups() {
             mac_count: 8,
             linux_count: 4,
             windows_count: 2,
+            unknown_count: 0,
         }
     );
 
@@ -1074,6 +1075,7 @@ async fn test_signups() {
             mac_count: 5,
             linux_count: 2,
             windows_count: 1,
+            unknown_count: 0,
         }
     );