[lib] introduce useDeleteFarcasterChannelTag
ClosedPublic
Actions

Authored by ginsu on Fri, Apr 26, 12:58 PM.

Details

Reviewers

atul
bartek
tomek

Commits

rCOMMa7783725d1fd: [lib] introduce useDeleteFarcasterChannelTag

Summary

This diff introduces useDeleteFarcasterChannelTag which is the actual action logic that we are running when we dispatch deleteFarcasterChannelTagActionTypes

Depends on D11815

Test Plan

Locally I was able to confirm that when I dispatched deleteFarcasterChannelTagActionTypes I would call the delete_farcaster_channel_tag endpoint and get the expected behavoir

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ginsu created this revision.Fri, Apr 26, 12:58 PM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptFri, Apr 26, 12:58 PM

ginsu edited the summary of this revision. (Show Details)Fri, Apr 26, 12:58 PM

ginsu added a parent revision: D11815: [lib] introduce deleteFarcasterChannelTagActionTypes.

ginsu added reviewers: atul, bartek, tomek.Fri, Apr 26, 1:10 PM

Harbormaster completed remote builds in B28544: Diff 39600.Fri, Apr 26, 1:30 PM

ginsu requested review of this revision.Fri, Apr 26, 1:30 PM

tomek accepted this revision.Mon, Apr 29, 1:31 AM

tomek added inline comments.

lib/actions/community-actions.js
86–88 ↗	(On Diff #39600)	It would be easier to review this and the last diff if they were merged together. They are tightly coupled, e.g. the payload of the successful action can't be verified in the last diff without this one. (But it doesn't make sense to merge them now.)