Differential D15501

[lib] Handle unpinning a message
ClosedPublic
Actions

Authored by tomek on Oct 22 2025, 2:07 PM.

Details

Reviewers

kamil

Commits

rCOMMf33b2a5236e0: [lib] Handle unpinning a message

Summary

Call the proper API method.

https://linear.app/comm/issue/ENG-11220/handle-message-pinning

Depends on D15500

Test Plan

Tested in combination with the rest of the stack: both on native and web, test if pinning and unpinning works. Also tested if the pins persist app reopens. Checked if the pins are correctly saved on Farcaster.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

tomek created this revision.Oct 22 2025, 2:07 PM

Herald added a subscriber: ashoat. · View Herald TranscriptOct 22 2025, 2:07 PM

tomek added a child revision: D15502: [lib] Optimistically update after pinning.Oct 22 2025, 2:11 PM

Rebase

Harbormaster failed remote builds in B35985: Diff 50727!Oct 22 2025, 3:28 PM

Harbormaster completed remote builds in B35991: Diff 50733.Oct 22 2025, 5:36 PM

tomek requested review of this revision.Oct 22 2025, 5:36 PM

kamil accepted this revision.Oct 23 2025, 1:20 PM

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

Closed by commit rCOMMf33b2a5236e0: [lib] Handle unpinning a message. · Explain WhyOct 23 2025, 4:11 PM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMMf33b2a5236e0: [lib] Handle unpinning a message.

Revision Contents
Changeset List

Path

Size

lib/

shared/

farcaster/

farcaster-api.js

15 lines

threads/

protocols/

farcaster-thread-protocol.js

3 lines

Diff 50727

View Options

lib/shared/farcaster/farcaster-api.js