Page MenuHomePhabricator
Differential D8028

[landing] key competitor data
ClosedPublic
Actions

Authored by ginsu on May 30 2023, 8:00 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 2, 3:58 AM
Unknown Object (File)
Sun, Mar 30, 12:01 PM
Unknown Object (File)
Sun, Mar 30, 7:35 AM
Unknown Object (File)
Sun, Mar 30, 3:07 AM
Unknown Object (File)
Sun, Mar 30, 2:18 AM
Unknown Object (File)
Sat, Mar 29, 8:21 PM
Unknown Object (File)
Fri, Mar 28, 11:32 PM
Unknown Object (File)
Wed, Mar 26, 12:08 AM
View All Files
Subscribers
ashoat
atul
tomek

Details

Reviewers
atul
kamil
Commits
rCOMM5de35fc7b603: [landing] key competitor data
Summary

In a subsequent diff we are going to want to access the competitor data by its key. This diff keys the competitorData so that we can do that

Depends on D8025

Test Plan

flow and this will be further tested in subsquent diffs

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ginsu created this revision.May 30 2023, 8:00 AM
Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptMay 30 2023, 8:00 AM
ginsu edited the summary of this revision. (Show Details)May 30 2023, 8:08 AM
ginsu added reviewers: atul, kamil.
ginsu added a parent revision: D8025: [landing] introduce feature modal component.
Harbormaster completed remote builds in B19903: Diff 27265.May 30 2023, 8:18 AM
ginsu requested review of this revision.May 30 2023, 8:18 AM
ginsu added a child revision: D8029: [landing] introduce Discord competitor data.May 30 2023, 8:56 AM
atul accepted this revision.May 30 2023, 8:58 AM

If we only plan on exporting keyedCompetitorData, could we just add keys to competitorData and turn it from array -> keyed obj?

This revision is now accepted and ready to land.May 30 2023, 8:58 AM
ginsu updated this revision to Diff 27271.May 30 2023, 11:00 AM

address comments

Harbormaster completed remote builds in B19909: Diff 27271.May 30 2023, 11:19 AM
atul added a comment.May 30 2023, 11:30 AM

(Looks like you added the Signal key in the update, did you mean to update the other entries as well?)

ginsu added a comment.May 30 2023, 11:32 AM

(Looks like you added the Signal key in the update, did you mean to update the other entries as well?)

Yes I will do that in each separate diff

ginsu updated this revision to Diff 29660.Aug 8 2023, 12:18 PM

rebase before landing

Harbormaster completed remote builds in B21620: Diff 29660.Aug 8 2023, 1:11 PM
Closed by commit rCOMM5de35fc7b603: [landing] key competitor data (authored by ginsu). · Explain WhyAug 8 2023, 2:08 PM
This revision was automatically updated to reflect the committed changes.
ginsu added a commit: rCOMM5de35fc7b603: [landing] key competitor data.

Revision Contents
Changeset List

PathSize
landing/
7 lines

Diff 27265

View Options

landing/competitor-data.js

Show First 20 Lines • Show All 62 Lines • ▼ Show 20 Linesconst entryQueryInputValidator: TInterface<EntryQueryInput> = tShape({
filters: t.maybe( filters: t.maybe(
t.list( t.list(
t.union([ t.union([
tShape({ tShape({
type: tString(calendarThreadFilterTypes.NOT_DELETED), type: tString(calendarThreadFilterTypes.NOT_DELETED),
}), }),
tShape({ tShape({
type: tString(calendarThreadFilterTypes.THREAD_LIST), type: tString(calendarThreadFilterTypes.THREAD_LIST),
threadIDs: t.list(t.String), threadIDs: t.list(tID),
}), }),
]), ]),
), ),
), ),
}); });
const newEntryQueryInputValidator: TInterface<CalendarQuery> = tShape({ const newEntryQueryInputValidator: TInterface<CalendarQuery> = tShape({
startDate: tDate, startDate: tDate,
endDate: tDate, endDate: tDate,
filters: t.list( filters: t.list(
t.union([ t.union([
tShape({ tShape({
type: tString(calendarThreadFilterTypes.NOT_DELETED), type: tString(calendarThreadFilterTypes.NOT_DELETED),
}), }),
tShape({ tShape({
type: tString(calendarThreadFilterTypes.THREAD_LIST), type: tString(calendarThreadFilterTypes.THREAD_LIST),
threadIDs: t.list(t.String), threadIDs: t.list(tID),
}), }),
]), ]),
), ),
}); });
function normalizeCalendarQuery(input: any): CalendarQuery { function normalizeCalendarQuery(input: any): CalendarQuery {
if (input.filters) { if (input.filters) {
return { return {
▲ Show 20 Lines • Show All 49 Lines • ▼ Show 20 Lines): Promise<FetchEntryInfosResponse> {
const response = await fetchEntryInfos(viewer, [request]); const response = await fetchEntryInfos(viewer, [request]);
return validateOutput(viewer, fetchEntryInfosResponseValidator, { return validateOutput(viewer, fetchEntryInfosResponseValidator, {
...response, ...response,
userInfos: {}, userInfos: {},
}); });
} }
const entryRevisionHistoryFetchInputValidator = tShape({ const entryRevisionHistoryFetchInputValidator = tShape({
id: t.String, id: tID,
}); });
export const fetchEntryRevisionInfosResultValidator: TInterface<FetchEntryRevisionInfosResult> = export const fetchEntryRevisionInfosResultValidator: TInterface<FetchEntryRevisionInfosResult> =
tShape<FetchEntryRevisionInfosResult>({ tShape<FetchEntryRevisionInfosResult>({
result: t.list(historyRevisionInfoValidator), result: t.list(historyRevisionInfoValidator),
}); });
async function entryRevisionFetchResponder( async function entryRevisionFetchResponder(
Show All 11 Lines): Promise<FetchEntryRevisionInfosResult> {
); );
} }
const createEntryRequestInputValidator = tShape({ const createEntryRequestInputValidator = tShape({
text: t.String, text: t.String,
sessionID: t.maybe(t.String), sessionID: t.maybe(t.String),
timestamp: t.Number, timestamp: t.Number,
date: tDate, date: tDate,
threadID: t.String, threadID: tID,
localID: t.maybe(t.String), localID: t.maybe(t.String),
calendarQuery: t.maybe(newEntryQueryInputValidator), calendarQuery: t.maybe(newEntryQueryInputValidator),
}); });
export const saveEntryResponseValidator: TInterface<SaveEntryResponse> = export const saveEntryResponseValidator: TInterface<SaveEntryResponse> =
tShape<SaveEntryResponse>({ tShape<SaveEntryResponse>({
entryID: tID, entryID: tID,
newMessageInfos: t.list(rawMessageInfoValidator), newMessageInfos: t.list(rawMessageInfoValidator),
updatesResult: serverCreateUpdatesResponseValidator, updatesResult: serverCreateUpdatesResponseValidator,
}); });
async function entryCreationResponder( async function entryCreationResponder(
viewer: Viewer, viewer: Viewer,
input: any, input: any,
): Promise<SaveEntryResponse> { ): Promise<SaveEntryResponse> {
const request: CreateEntryRequest = input; const request: CreateEntryRequest = input;
await validateInput(viewer, createEntryRequestInputValidator, request); await validateInput(viewer, createEntryRequestInputValidator, request);
const response = await createEntry(viewer, request); const response = await createEntry(viewer, request);
return validateOutput(viewer, saveEntryResponseValidator, response); return validateOutput(viewer, saveEntryResponseValidator, response);
} }
const saveEntryRequestInputValidator = tShape({ const saveEntryRequestInputValidator = tShape({
entryID: t.String, entryID: tID,
text: t.String, text: t.String,
prevText: t.String, prevText: t.String,
sessionID: t.maybe(t.String), sessionID: t.maybe(t.String),
timestamp: t.Number, timestamp: t.Number,
calendarQuery: t.maybe(newEntryQueryInputValidator), calendarQuery: t.maybe(newEntryQueryInputValidator),
}); });
async function entryUpdateResponder( async function entryUpdateResponder(
viewer: Viewer, viewer: Viewer,
input: any, input: any,
): Promise<SaveEntryResponse> { ): Promise<SaveEntryResponse> {
const request: SaveEntryRequest = input; const request: SaveEntryRequest = input;
await validateInput(viewer, saveEntryRequestInputValidator, request); await validateInput(viewer, saveEntryRequestInputValidator, request);
const response = await updateEntry(viewer, request); const response = await updateEntry(viewer, request);
return validateOutput(viewer, saveEntryResponseValidator, response); return validateOutput(viewer, saveEntryResponseValidator, response);
} }
const deleteEntryRequestInputValidator = tShape({ const deleteEntryRequestInputValidator = tShape({
entryID: t.String, entryID: tID,
prevText: t.String, prevText: t.String,
sessionID: t.maybe(t.String), sessionID: t.maybe(t.String),
timestamp: t.Number, timestamp: t.Number,
calendarQuery: t.maybe(newEntryQueryInputValidator), calendarQuery: t.maybe(newEntryQueryInputValidator),
}); });
export const deleteEntryResponseValidator: TInterface<DeleteEntryResponse> = export const deleteEntryResponseValidator: TInterface<DeleteEntryResponse> =
tShape<DeleteEntryResponse>({ tShape<DeleteEntryResponse>({
newMessageInfos: t.list(rawMessageInfoValidator), newMessageInfos: t.list(rawMessageInfoValidator),
threadID: tID, threadID: tID,
updatesResult: serverCreateUpdatesResponseValidator, updatesResult: serverCreateUpdatesResponseValidator,
}); });
async function entryDeletionResponder( async function entryDeletionResponder(
viewer: Viewer, viewer: Viewer,
input: any, input: any,
): Promise<DeleteEntryResponse> { ): Promise<DeleteEntryResponse> {
const request: DeleteEntryRequest = input; const request: DeleteEntryRequest = input;
await validateInput(viewer, deleteEntryRequestInputValidator, request); await validateInput(viewer, deleteEntryRequestInputValidator, request);
const response = await deleteEntry(viewer, request); const response = await deleteEntry(viewer, request);
return validateOutput(viewer, deleteEntryResponseValidator, response); return validateOutput(viewer, deleteEntryResponseValidator, response);
} }
const restoreEntryRequestInputValidator = tShape({ const restoreEntryRequestInputValidator = tShape({
entryID: t.String, entryID: tID,
sessionID: t.maybe(t.String), sessionID: t.maybe(t.String),
timestamp: t.Number, timestamp: t.Number,
calendarQuery: t.maybe(newEntryQueryInputValidator), calendarQuery: t.maybe(newEntryQueryInputValidator),
}); });
export const restoreEntryResponseValidator: TInterface<RestoreEntryResponse> = export const restoreEntryResponseValidator: TInterface<RestoreEntryResponse> =
tShape<RestoreEntryResponse>({ tShape<RestoreEntryResponse>({
newMessageInfos: t.list(rawMessageInfoValidator), newMessageInfos: t.list(rawMessageInfoValidator),
▲ Show 20 Lines • Show All 61 Lines • Show Last 20 Lines