Page MenuHomePhorge
Files F32162534

D15382.1765045061.diff
No OneTemporary
Actions

Size
2 KB
Referenced Files
None
Subscribers
None

D15382.1765045061.diff
View Options

diff --git a/services/tunnelbroker/src/token_distributor/notif_utils.rs b/services/tunnelbroker/src/token_distributor/notif_utils.rs
--- a/services/tunnelbroker/src/token_distributor/notif_utils.rs
+++ b/services/tunnelbroker/src/token_distributor/notif_utils.rs
@@ -1,3 +1,5 @@
+use std::borrow::Cow;
+
use tunnelbroker_messages::farcaster::{
DirectCastConversation, DirectCastMessageType,
RefreshDirectCastConversationPayload,
@@ -8,7 +10,7 @@
pub fn prepare_notif_payload(
payload: &RefreshDirectCastConversationPayload,
conversation: &DirectCastConversation,
- recipient_fid: Option<&String>,
+ recipient_fid: &str,
) -> Option<GenericNotifPayload> {
let RefreshDirectCastConversationPayload {
conversation_id,
@@ -17,7 +19,6 @@
} = payload;
if conversation.muted {
- // TODO: badge only?
return None;
}
if message.message_type != DirectCastMessageType::Text {
@@ -25,7 +26,7 @@
}
// Don't send a notif from self
- if recipient_fid.is_some_and(|fid| *fid == message.sender_fid.to_string()) {
+ if recipient_fid == message.sender_fid.to_string() {
return None;
}
@@ -35,27 +36,30 @@
let has_videos =
message_metadata.is_some_and(|metadata| metadata["videos"].is_array());
+ let sender_name = conversation
+ .participant(message.sender_fid)
+ .map(|u| u.display_name.as_str());
let title = conversation
.name
.as_deref()
- .or_else(|| {
- conversation
- .participant(message.sender_fid)
- .map(|u| u.display_name.as_str())
- })
- .unwrap_or("Farcaster");
+ .or(sender_name)
+ .unwrap_or("Farcaster DC");
- let body = if has_photos {
- "[Photo message]"
+ let body: Cow<str> = if has_photos {
+ sender_name
+ .map(|author| format!("{author} sent a photo").into())
+ .unwrap_or("[Photo message]".into())
} else if has_videos {
- "[Video message]"
+ sender_name
+ .map(|author| format!("{author} sent a video").into())
+ .unwrap_or("[Video message]".into())
} else {
- message.message.as_str()
+ Cow::Borrowed(message.message.as_str())
};
Some(GenericNotifPayload {
title: trim_text(title, 100),
- body: trim_text(body, 300),
+ body: trim_text(&body, 300),
thread_id: format!("FARCASTER#{}", conversation_id),
})
}
diff --git a/services/tunnelbroker/src/token_distributor/token_connection.rs b/services/tunnelbroker/src/token_distributor/token_connection.rs
--- a/services/tunnelbroker/src/token_distributor/token_connection.rs
+++ b/services/tunnelbroker/src/token_distributor/token_connection.rs
@@ -528,11 +528,8 @@
"Failed to fetch conversation details: {e:?}",
))
})?;
- let notif = prepare_notif_payload(
- payload,
- &conversation,
- Some(&self.token_info.fid),
- );
+ let notif =
+ prepare_notif_payload(payload, &conversation, &self.token_info.fid);
Ok::<_, TokenConnectionError>(notif)
};

File Metadata

Mime Type
text/plain
Expires
Sat, Dec 6, 6:17 PM (15 h, 47 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
5840069
Default Alt Text
D15382.1765045061.diff (2 KB)

Event Timeline