Page MenuHomePhabricator
Differential D11814

[keyserver/lib] introduce delete_farcaster_channel_tag endpoint
ClosedPublic
Actions

Authored by ginsu on Apr 26 2024, 12:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 22, 4:07 PM
Unknown Object (File)
Tue, Mar 11, 1:09 PM
Unknown Object (File)
Fri, Mar 7, 4:01 AM
Unknown Object (File)
Tue, Mar 4, 9:08 PM
Unknown Object (File)
Mon, Mar 3, 12:33 AM
Unknown Object (File)
Mon, Mar 3, 12:33 AM
Unknown Object (File)
Feb 14 2025, 3:09 PM
Unknown Object (File)
Jan 23 2025, 5:17 AM
View All 86 Files
Subscribers
ashoat
tomek

Details

Reviewers
atul
bartek
tomek
Commits
rCOMMffc0c01586ef: [keyserver/lib] introduce delete_farcaster_channel_tag endpoint
Summary

This diff introduces the delete_farcaster_channel_tag endpoint so that we can call the deleteFarcasterChannelTag we just introduced

Depends on D11813

Test Plan

In my local stack I was able to call the delete_farcaster_channel_tag endpoint and use the deleteFarcasterChannelTag from the client

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ginsu created this revision.Apr 26 2024, 12:43 PM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptApr 26 2024, 12:43 PM
ginsu added inline comments.Apr 26 2024, 12:44 PM
keyserver/src/responders/farcaster-channel-tag-responders.js
39 ↗(On Diff #39593)

In my next project, we will refactor this so the blob holder is no longer part of the request

ginsu edited the summary of this revision. (Show Details)Apr 26 2024, 12:45 PM
ginsu added reviewers: atul, bartek, tomek.
ginsu added a parent revision: D11813: [keyserver] introduce deleteFarcasterChannelTag.
ginsu added a child revision: D11815: [lib] introduce deleteFarcasterChannelTagActionTypes.Apr 26 2024, 12:48 PM
Harbormaster completed remote builds in B28537: Diff 39593.Apr 26 2024, 12:58 PM
ginsu requested review of this revision.Apr 26 2024, 12:58 PM
tomek accepted this revision.Apr 29 2024, 1:22 AM
tomek added inline comments.
keyserver/src/responders/farcaster-channel-tag-responders.js
35–52 ↗(On Diff #39593)

We should maintain consistency of our exports

This revision is now accepted and ready to land.Apr 29 2024, 1:22 AM
ginsu updated this revision to Diff 39729.May 1 2024, 10:42 PM

address comments + rebase before landing

This revision was landed with ongoing or failed builds.May 1 2024, 10:52 PM
Closed by commit rCOMMffc0c01586ef: [keyserver/lib] introduce delete_farcaster_channel_tag endpoint (authored by ginsu). · Explain Why
This revision was automatically updated to reflect the committed changes.
ginsu added a commit: rCOMMffc0c01586ef: [keyserver/lib] introduce delete_farcaster_channel_tag endpoint.
Harbormaster completed remote builds in B28622: Diff 39729.May 1 2024, 10:58 PM

Revision Contents
Changeset List

PathSize
keyserver/
src/
8 lines
responders/
28 lines
lib/
types/
1 line

Diff 39734

View Options

keyserver/src/endpoints.js

Show All 22 Lines
import type { import type {
UpdateUserAvatarRequest, UpdateUserAvatarRequest,
UpdateUserAvatarResponse, UpdateUserAvatarResponse,
} from './avatar-types.js'; } from './avatar-types.js';
import type { import type {
CommunityStore, CommunityStore,
AddCommunityPayload, AddCommunityPayload,
CreateOrUpdateFarcasterChannelTagResponse, CreateOrUpdateFarcasterChannelTagResponse,
DeleteFarcasterChannelTagPayload,
} from './community-types.js'; } from './community-types.js';
import type { MessageSourceMetadata, DBOpsStore } from './db-ops-types.js'; import type { MessageSourceMetadata, DBOpsStore } from './db-ops-types.js';
import type { import type {
GetVersionActionPayload, GetVersionActionPayload,
LastCommunicatedPlatformDetails, LastCommunicatedPlatformDetails,
} from './device-types.js'; } from './device-types.js';
import type { DraftStore } from './draft-types.js'; import type { DraftStore } from './draft-types.js';
import type { EnabledApps, SupportedApps } from './enabled-apps.js'; import type { EnabledApps, SupportedApps } from './enabled-apps.js';
▲ Show 20 Lines • Show All 1,367 Lines • ▼ Show 20 Lines | {
+payload: CreateOrUpdateFarcasterChannelTagResponse, +payload: CreateOrUpdateFarcasterChannelTagResponse,
+loadingInfo: LoadingInfo, +loadingInfo: LoadingInfo,
} }
| { | {
+type: 'CREATE_OR_UPDATE_FARCASTER_CHANNEL_TAG_FAILED', +type: 'CREATE_OR_UPDATE_FARCASTER_CHANNEL_TAG_FAILED',
+error: true, +error: true,
+payload: Error, +payload: Error,
+loadingInfo: LoadingInfo, +loadingInfo: LoadingInfo,
}
| {
+type: 'DELETE_FARCASTER_CHANNEL_TAG_STARTED',
+loadingInfo?: LoadingInfo,
+payload?: void,
}
| {
+type: 'DELETE_FARCASTER_CHANNEL_TAG_SUCCESS',
+payload: DeleteFarcasterChannelTagPayload,
+loadingInfo: LoadingInfo,
}
| {
+type: 'DELETE_FARCASTER_CHANNEL_TAG_FAILED',
+error: true,
+payload: Error,
+loadingInfo: LoadingInfo,
}, },
}>; }>;
export type ActionPayload = ?(Object | Array<*> | $ReadOnlyArray<*> | string); export type ActionPayload = ?(Object | Array<*> | $ReadOnlyArray<*> | string);
export type DispatchSource = 'tunnelbroker' | 'tab-sync'; export type DispatchSource = 'tunnelbroker' | 'tab-sync';
export type SuperAction = { export type SuperAction = {
+type: string, +type: string,
+payload?: ActionPayload, +payload?: ActionPayload,
Show All 15 Lines
View Options

keyserver/src/responders/farcaster-channel-tag-responders.js

Show All 22 Lines
import type { import type {
UpdateUserAvatarRequest, UpdateUserAvatarRequest,
UpdateUserAvatarResponse, UpdateUserAvatarResponse,
} from './avatar-types.js'; } from './avatar-types.js';
import type { import type {
CommunityStore, CommunityStore,
AddCommunityPayload, AddCommunityPayload,
CreateOrUpdateFarcasterChannelTagResponse, CreateOrUpdateFarcasterChannelTagResponse,
DeleteFarcasterChannelTagPayload,
} from './community-types.js'; } from './community-types.js';
import type { MessageSourceMetadata, DBOpsStore } from './db-ops-types.js'; import type { MessageSourceMetadata, DBOpsStore } from './db-ops-types.js';
import type { import type {
GetVersionActionPayload, GetVersionActionPayload,
LastCommunicatedPlatformDetails, LastCommunicatedPlatformDetails,
} from './device-types.js'; } from './device-types.js';
import type { DraftStore } from './draft-types.js'; import type { DraftStore } from './draft-types.js';
import type { EnabledApps, SupportedApps } from './enabled-apps.js'; import type { EnabledApps, SupportedApps } from './enabled-apps.js';
▲ Show 20 Lines • Show All 1,367 Lines • ▼ Show 20 Lines | {
+payload: CreateOrUpdateFarcasterChannelTagResponse, +payload: CreateOrUpdateFarcasterChannelTagResponse,
+loadingInfo: LoadingInfo, +loadingInfo: LoadingInfo,
} }
| { | {
+type: 'CREATE_OR_UPDATE_FARCASTER_CHANNEL_TAG_FAILED', +type: 'CREATE_OR_UPDATE_FARCASTER_CHANNEL_TAG_FAILED',
+error: true, +error: true,
+payload: Error, +payload: Error,
+loadingInfo: LoadingInfo, +loadingInfo: LoadingInfo,
}
| {
+type: 'DELETE_FARCASTER_CHANNEL_TAG_STARTED',
+loadingInfo?: LoadingInfo,
+payload?: void,
}
| {
+type: 'DELETE_FARCASTER_CHANNEL_TAG_SUCCESS',
+payload: DeleteFarcasterChannelTagPayload,
+loadingInfo: LoadingInfo,
}
| {
+type: 'DELETE_FARCASTER_CHANNEL_TAG_FAILED',
+error: true,
+payload: Error,
+loadingInfo: LoadingInfo,
}, },
}>; }>;
export type ActionPayload = ?(Object | Array<*> | $ReadOnlyArray<*> | string); export type ActionPayload = ?(Object | Array<*> | $ReadOnlyArray<*> | string);
export type DispatchSource = 'tunnelbroker' | 'tab-sync'; export type DispatchSource = 'tunnelbroker' | 'tab-sync';
export type SuperAction = { export type SuperAction = {
+type: string, +type: string,
+payload?: ActionPayload, +payload?: ActionPayload,
Show All 15 Lines
View Options

lib/types/endpoints.js

Show All 22 Lines
import type { import type {
UpdateUserAvatarRequest, UpdateUserAvatarRequest,
UpdateUserAvatarResponse, UpdateUserAvatarResponse,
} from './avatar-types.js'; } from './avatar-types.js';
import type { import type {
CommunityStore, CommunityStore,
AddCommunityPayload, AddCommunityPayload,
CreateOrUpdateFarcasterChannelTagResponse, CreateOrUpdateFarcasterChannelTagResponse,
DeleteFarcasterChannelTagPayload,
} from './community-types.js'; } from './community-types.js';
import type { MessageSourceMetadata, DBOpsStore } from './db-ops-types.js'; import type { MessageSourceMetadata, DBOpsStore } from './db-ops-types.js';
import type { import type {
GetVersionActionPayload, GetVersionActionPayload,
LastCommunicatedPlatformDetails, LastCommunicatedPlatformDetails,
} from './device-types.js'; } from './device-types.js';
import type { DraftStore } from './draft-types.js'; import type { DraftStore } from './draft-types.js';
import type { EnabledApps, SupportedApps } from './enabled-apps.js'; import type { EnabledApps, SupportedApps } from './enabled-apps.js';
▲ Show 20 Lines • Show All 1,367 Lines • ▼ Show 20 Lines | {
+payload: CreateOrUpdateFarcasterChannelTagResponse, +payload: CreateOrUpdateFarcasterChannelTagResponse,
+loadingInfo: LoadingInfo, +loadingInfo: LoadingInfo,
} }
| { | {
+type: 'CREATE_OR_UPDATE_FARCASTER_CHANNEL_TAG_FAILED', +type: 'CREATE_OR_UPDATE_FARCASTER_CHANNEL_TAG_FAILED',
+error: true, +error: true,
+payload: Error, +payload: Error,
+loadingInfo: LoadingInfo, +loadingInfo: LoadingInfo,
}
| {
+type: 'DELETE_FARCASTER_CHANNEL_TAG_STARTED',
+loadingInfo?: LoadingInfo,
+payload?: void,
}
| {
+type: 'DELETE_FARCASTER_CHANNEL_TAG_SUCCESS',
+payload: DeleteFarcasterChannelTagPayload,
+loadingInfo: LoadingInfo,
}
| {
+type: 'DELETE_FARCASTER_CHANNEL_TAG_FAILED',
+error: true,
+payload: Error,
+loadingInfo: LoadingInfo,
}, },
}>; }>;
export type ActionPayload = ?(Object | Array<*> | $ReadOnlyArray<*> | string); export type ActionPayload = ?(Object | Array<*> | $ReadOnlyArray<*> | string);
export type DispatchSource = 'tunnelbroker' | 'tab-sync'; export type DispatchSource = 'tunnelbroker' | 'tab-sync';
export type SuperAction = { export type SuperAction = {
+type: string, +type: string,
+payload?: ActionPayload, +payload?: ActionPayload,
Show All 15 Lines