From 270ce539bf3ebbd85a7929edacbc9ffb52e5ea26 Mon Sep 17 00:00:00 2001 From: Dessalines Date: Mon, 15 Mar 2021 18:18:50 -0400 Subject: [PATCH] Removing some TODOS. --- crates/api/src/lib.rs | 9 --------- crates/api_structs/src/lib.rs | 2 -- 2 files changed, 11 deletions(-) diff --git a/crates/api/src/lib.rs b/crates/api/src/lib.rs index 21afcee24..aebabde8a 100644 --- a/crates/api/src/lib.rs +++ b/crates/api/src/lib.rs @@ -72,15 +72,6 @@ pub(crate) async fn is_mod_or_admin( Ok(()) } -// TODO this probably isn't necessary anymore -// pub async fn is_admin(pool: &DbPool, person_id: i32) -> Result<(), LemmyError> { -// let user = blocking(pool, move |conn| LocalUser::read(conn, person_id)).await??; -// if !user.admin { -// return Err(ApiError::err("not_an_admin").into()); -// } -// Ok(()) -// } - pub fn is_admin(local_user_view: &LocalUserView) -> Result<(), LemmyError> { if !local_user_view.local_user.admin { return Err(ApiError::err("not_an_admin").into()); diff --git a/crates/api_structs/src/lib.rs b/crates/api_structs/src/lib.rs index e0fab6b24..e25b05791 100644 --- a/crates/api_structs/src/lib.rs +++ b/crates/api_structs/src/lib.rs @@ -68,7 +68,6 @@ pub async fn send_local_notifs( Ok(ids) } -// TODO should this really use person_ids as recipient ids? or local_user_ids ? fn do_send_local_notifs( conn: &PgConnection, mentions: &[MentionData], @@ -85,7 +84,6 @@ fn do_send_local_notifs( .filter(|m| m.is_local() && m.name.ne(&person.name)) .collect::>() { - // TODO do a local user fetch if let Ok(mention_user_view) = LocalUserView::read_from_name(&conn, &mention.name) { // TODO // At some point, make it so you can't tag the parent creator either