Differential D15483

[tunnelbroker] Prefer username in Farcaster notifs
ClosedPublic
Actions

Authored by tomek on Oct 16 2025, 12:49 PM.

Details

Reviewers

kamil

Commits

rCOMM47b3422ebaef: [tunnelbroker] Prefer username in Farcaster notifs

Summary

Prefer the username just like in the app.

https://linear.app/comm/issue/ENG-11462/we-should-use-farcaster-username-instead-of-name-in-notifs

Test Plan

Add some logs to check if the username was used.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Oct 16 2025, 12:49 PM

Herald added a subscriber: ashoat. · View Herald TranscriptOct 16 2025, 12:49 PM

tomek published this revision for review.Oct 16 2025, 12:53 PM

Harbormaster completed remote builds in B35956: Diff 50681.Oct 16 2025, 1:10 PM

kamil accepted this revision.Oct 16 2025, 1:49 PM

This revision is now accepted and ready to land.Oct 16 2025, 1:49 PM

Closed by commit rCOMM47b3422ebaef: [tunnelbroker] Prefer username in Farcaster notifs. · Explain WhyOct 16 2025, 3:22 PM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM47b3422ebaef: [tunnelbroker] Prefer username in Farcaster notifs.

Revision Contents
Changeset List

Path

Size

services/

tunnelbroker/

src/

token_distributor/

notif_utils.rs

2 lines

shared/

tunnelbroker_messages/

src/

messages/

farcaster.rs

2 lines

Diff 50686

View Options

services/tunnelbroker/src/token_distributor/notif_utils.rs