diff --git a/crates/collab/src/db/queries/shared_threads.rs b/crates/collab/src/db/queries/shared_threads.rs index 2a02b247eb741e..9cb320d45d46bc 100644 --- a/crates/collab/src/db/queries/shared_threads.rs +++ b/crates/collab/src/db/queries/shared_threads.rs @@ -64,11 +64,10 @@ impl Database { return Ok(None); }; - let user = user::Entity::find_by_id(thread.user_id).one(&*tx).await?; - - let username = user - .map(|u| u.github_login) - .unwrap_or_else(|| "Unknown".to_string()); + // We can no longer read the `github_login` from Collab. + // + // Opting to just use "Unknown" here, as the feature is staff-only and infrequently used. + let username = "Unknown".to_string(); Ok(Some((thread, username))) }) diff --git a/crates/collab/src/db/tables/user.rs b/crates/collab/src/db/tables/user.rs index 75dd72f1c122d9..3a0b7ad745de94 100644 --- a/crates/collab/src/db/tables/user.rs +++ b/crates/collab/src/db/tables/user.rs @@ -8,7 +8,9 @@ use serde::Serialize; pub struct Model { #[sea_orm(primary_key)] pub id: UserId, + #[cfg(feature = "test-support")] pub github_login: String, + #[cfg(feature = "test-support")] pub github_user_id: i32, pub admin: bool, pub connected_once: bool, diff --git a/crates/collab/tests/integration/db_tests/db_tests.rs b/crates/collab/tests/integration/db_tests/db_tests.rs index 17123db1b41035..dc3faae5cdd50f 100644 --- a/crates/collab/tests/integration/db_tests/db_tests.rs +++ b/crates/collab/tests/integration/db_tests/db_tests.rs @@ -285,7 +285,7 @@ async fn test_upsert_shared_thread(db: &Arc) { assert_eq!(thread.title, title); assert_eq!(thread.data, data); assert_eq!(thread.user_id, user_id); - assert_eq!(username, "user1"); + assert_eq!(username, "Unknown"); } test_both_dbs!(