[lib] Fix FID from userID extraction
ClosedPublic
Actions

Authored by bartek on Sep 3 2025, 12:39 PM.

Details

Reviewers

kamil
tomek

Commits

rCOMMe87aa2bb989f: [lib] Fix FID from userID extraction

Summary

I accidently used the wrong function. The ID should begin with this prefix.
Otherwise, pending thread IDs were treated as FIDs when they contained Farcaster users

Test Plan

Thread IDs are no longer tried to have username fetched

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Sep 3 2025, 12:39 PM

bartek held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptSep 3 2025, 12:39 PM

Harbormaster failed remote builds in B35624: Diff 50151!Sep 3 2025, 12:41 PM

bartek published this revision for review.Sep 3 2025, 2:19 PM

kamil accepted this revision.Sep 3 2025, 3:56 PM

This revision is now accepted and ready to land.Sep 3 2025, 3:56 PM

Closed by commit rCOMMe87aa2bb989f: [lib] Fix FID from userID extraction. · Explain WhySep 4 2025, 10:04 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMe87aa2bb989f: [lib] Fix FID from userID extraction.

Revision Contents
Changeset List

Path

Size

lib/

shared/

id-utils.js

2 lines

Diff 50151

View Options

[lib] Fix FID from userID extractionClosedPublicActions