Page MenuHomePhabricator
Differential D13140

[web] Create a function that fetches messages in batches
ClosedPublic
Actions

Authored by tomek on Aug 22 2024, 10:19 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 8, 5:10 PM
Unknown Object (File)
Sat, Mar 8, 5:10 PM
Unknown Object (File)
Wed, Mar 5, 6:09 AM
Unknown Object (File)
Feb 7 2025, 3:14 PM
Unknown Object (File)
Jan 30 2025, 2:55 PM
Unknown Object (File)
Jan 19 2025, 5:52 PM
Unknown Object (File)
Jan 16 2025, 8:04 PM
Unknown Object (File)
Jan 15 2025, 11:10 AM
View All 45 Files
Subscribers
ashoat

Details

Reviewers
kamil
inka
Commits
rCOMM1cee5c5f3b22: [web] Create a function that fetches messages in batches
Summary

The function uses limit and offset to get a batch of messages.

https://linear.app/comm/issue/ENG-8705/create-a-function-that-fetches-more-messages-and-dispatches-the-new

Depends on D13138

Test Plan

Tests.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

tomek created this revision.Aug 22 2024, 10:19 AM
Herald added a subscriber: ashoat. · View Herald TranscriptAug 22 2024, 10:19 AM
Harbormaster completed remote builds in B31220: Diff 43591.Aug 22 2024, 10:37 AM
tomek requested review of this revision.Aug 22 2024, 10:37 AM
tomek added a child revision: D13141: [native] Expose messages fetching function through JSI.Aug 22 2024, 10:40 AM
tomek updated this revision to Diff 43619.EditedAug 23 2024, 9:09 AM

Extract thick thread types

Harbormaster completed remote builds in B31246: Diff 43619.Aug 23 2024, 9:26 AM
tomek updated this revision to Diff 43622.Aug 23 2024, 9:26 AM

Rebase

Harbormaster completed remote builds in B31249: Diff 43622.Aug 23 2024, 9:43 AM
kamil accepted this revision.Aug 26 2024, 2:43 AM
kamil added inline comments.
native/cpp/CommonCpp/DatabaseManagers/DatabaseQueryExecutor.h
197–198

This started to be annoying, thinking how long it could take to prioritize ENG-5719 after D12736 is landed

web/shared-worker/queries/fetch-messages-queries.test.js
36

I would move 124 to const

This revision is now accepted and ready to land.Aug 26 2024, 2:43 AM
tomek updated this revision to Diff 43670.Aug 27 2024, 4:40 AM

Extract a constant

Harbormaster completed remote builds in B31278: Diff 43670.Aug 27 2024, 4:57 AM
Closed by commit rCOMM1cee5c5f3b22: [web] Create a function that fetches messages in batches (authored by tomek). · Explain WhyAug 27 2024, 8:02 AM
This revision was automatically updated to reflect the committed changes.
tomek added a commit: rCOMM1cee5c5f3b22: [web] Create a function that fetches messages in batches.

Revision Contents
Changeset List

PathSize
native/
cpp/
CommonCpp/
DatabaseManagers/
4 lines
4 lines
33 lines
web/
cpp/
3 lines
shared-worker/
_generated/
queries/
108 lines
types/
1 line
7 lines

Diff 43622

View Options

native/cpp/CommonCpp/DatabaseManagers/DatabaseQueryExecutor.h

// @flow // @flow
import type { import type {
FetchInviteLinksResponse, FetchInviteLinksResponse,
InviteLinkVerificationRequest, InviteLinkVerificationRequest,
InviteLinkVerificationResponse, InviteLinkVerificationResponse,
CreateOrUpdatePublicLinkRequest, CreateOrUpdatePublicLinkRequest,
InviteLink, InviteLink,
DisableInviteLinkRequest,
DisableInviteLinkPayload,
} from '../types/link-types.js'; } from '../types/link-types.js';
import type { CallServerEndpoint } from '../utils/call-server-endpoint.js'; import type { CallServerEndpoint } from '../utils/call-server-endpoint.js';
const verifyInviteLinkActionTypes = Object.freeze({ const verifyInviteLinkActionTypes = Object.freeze({
started: 'VERIFY_INVITE_LINK_STARTED', started: 'VERIFY_INVITE_LINK_STARTED',
success: 'VERIFY_INVITE_LINK_SUCCESS', success: 'VERIFY_INVITE_LINK_SUCCESS',
failed: 'VERIFY_INVITE_LINK_FAILED', failed: 'VERIFY_INVITE_LINK_FAILED',
}); });
▲ Show 20 Lines • Show All 53 Lines • ▼ Show 20 Lines return {
role: response.role, role: response.role,
communityID: response.communityID, communityID: response.communityID,
expirationTime: response.expirationTime, expirationTime: response.expirationTime,
limitOfUses: response.limitOfUses, limitOfUses: response.limitOfUses,
numberOfUses: response.numberOfUses, numberOfUses: response.numberOfUses,
}; };
}; };
const disableInviteLinkLinkActionTypes = Object.freeze({
started: 'DISABLE_INVITE_LINK_STARTED',
success: 'DISABLE_INVITE_LINK_SUCCESS',
failed: 'DISABLE_INVITE_LINK_FAILED',
});
const disableInviteLink =
(
callServerEndpoint: CallServerEndpoint,
): ((
request: DisableInviteLinkRequest,
) => Promise<DisableInviteLinkPayload>) =>
async request => {
await callServerEndpoint('disable_invite_link', request);
return {
name: request.name,
communityID: request.communityID,
};
};
export { export {
verifyInviteLinkActionTypes, verifyInviteLinkActionTypes,
verifyInviteLink, verifyInviteLink,
fetchPrimaryInviteLinkActionTypes, fetchPrimaryInviteLinkActionTypes,
fetchPrimaryInviteLinks, fetchPrimaryInviteLinks,
createOrUpdatePublicLinkActionTypes, createOrUpdatePublicLinkActionTypes,
createOrUpdatePublicLink, createOrUpdatePublicLink,
disableInviteLinkLinkActionTypes,
disableInviteLink,
}; };
View Options

native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.h

// @flow // @flow
import type { import type {
FetchInviteLinksResponse, FetchInviteLinksResponse,
InviteLinkVerificationRequest, InviteLinkVerificationRequest,
InviteLinkVerificationResponse, InviteLinkVerificationResponse,
CreateOrUpdatePublicLinkRequest, CreateOrUpdatePublicLinkRequest,
InviteLink, InviteLink,
DisableInviteLinkRequest,
DisableInviteLinkPayload,
} from '../types/link-types.js'; } from '../types/link-types.js';
import type { CallServerEndpoint } from '../utils/call-server-endpoint.js'; import type { CallServerEndpoint } from '../utils/call-server-endpoint.js';
const verifyInviteLinkActionTypes = Object.freeze({ const verifyInviteLinkActionTypes = Object.freeze({
started: 'VERIFY_INVITE_LINK_STARTED', started: 'VERIFY_INVITE_LINK_STARTED',
success: 'VERIFY_INVITE_LINK_SUCCESS', success: 'VERIFY_INVITE_LINK_SUCCESS',
failed: 'VERIFY_INVITE_LINK_FAILED', failed: 'VERIFY_INVITE_LINK_FAILED',
}); });
▲ Show 20 Lines • Show All 53 Lines • ▼ Show 20 Lines return {
role: response.role, role: response.role,
communityID: response.communityID, communityID: response.communityID,
expirationTime: response.expirationTime, expirationTime: response.expirationTime,
limitOfUses: response.limitOfUses, limitOfUses: response.limitOfUses,
numberOfUses: response.numberOfUses, numberOfUses: response.numberOfUses,
}; };
}; };
const disableInviteLinkLinkActionTypes = Object.freeze({
started: 'DISABLE_INVITE_LINK_STARTED',
success: 'DISABLE_INVITE_LINK_SUCCESS',
failed: 'DISABLE_INVITE_LINK_FAILED',
});
const disableInviteLink =
(
callServerEndpoint: CallServerEndpoint,
): ((
request: DisableInviteLinkRequest,
) => Promise<DisableInviteLinkPayload>) =>
async request => {
await callServerEndpoint('disable_invite_link', request);
return {
name: request.name,
communityID: request.communityID,
};
};
export { export {
verifyInviteLinkActionTypes, verifyInviteLinkActionTypes,
verifyInviteLink, verifyInviteLink,
fetchPrimaryInviteLinkActionTypes, fetchPrimaryInviteLinkActionTypes,
fetchPrimaryInviteLinks, fetchPrimaryInviteLinks,
createOrUpdatePublicLinkActionTypes, createOrUpdatePublicLinkActionTypes,
createOrUpdatePublicLink, createOrUpdatePublicLink,
disableInviteLinkLinkActionTypes,
disableInviteLink,
}; };
View Options

native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.cpp

// @flow // @flow
import type { import type {
FetchInviteLinksResponse, FetchInviteLinksResponse,
InviteLinkVerificationRequest, InviteLinkVerificationRequest,
InviteLinkVerificationResponse, InviteLinkVerificationResponse,
CreateOrUpdatePublicLinkRequest, CreateOrUpdatePublicLinkRequest,
InviteLink, InviteLink,
DisableInviteLinkRequest,
DisableInviteLinkPayload,
} from '../types/link-types.js'; } from '../types/link-types.js';
import type { CallServerEndpoint } from '../utils/call-server-endpoint.js'; import type { CallServerEndpoint } from '../utils/call-server-endpoint.js';
const verifyInviteLinkActionTypes = Object.freeze({ const verifyInviteLinkActionTypes = Object.freeze({
started: 'VERIFY_INVITE_LINK_STARTED', started: 'VERIFY_INVITE_LINK_STARTED',
success: 'VERIFY_INVITE_LINK_SUCCESS', success: 'VERIFY_INVITE_LINK_SUCCESS',
failed: 'VERIFY_INVITE_LINK_FAILED', failed: 'VERIFY_INVITE_LINK_FAILED',
}); });
▲ Show 20 Lines • Show All 53 Lines • ▼ Show 20 Lines return {
role: response.role, role: response.role,
communityID: response.communityID, communityID: response.communityID,
expirationTime: response.expirationTime, expirationTime: response.expirationTime,
limitOfUses: response.limitOfUses, limitOfUses: response.limitOfUses,
numberOfUses: response.numberOfUses, numberOfUses: response.numberOfUses,
}; };
}; };
const disableInviteLinkLinkActionTypes = Object.freeze({
started: 'DISABLE_INVITE_LINK_STARTED',
success: 'DISABLE_INVITE_LINK_SUCCESS',
failed: 'DISABLE_INVITE_LINK_FAILED',
});
const disableInviteLink =
(
callServerEndpoint: CallServerEndpoint,
): ((
request: DisableInviteLinkRequest,
) => Promise<DisableInviteLinkPayload>) =>
async request => {
await callServerEndpoint('disable_invite_link', request);
return {
name: request.name,
communityID: request.communityID,
};
};
export { export {
verifyInviteLinkActionTypes, verifyInviteLinkActionTypes,
verifyInviteLink, verifyInviteLink,
fetchPrimaryInviteLinkActionTypes, fetchPrimaryInviteLinkActionTypes,
fetchPrimaryInviteLinks, fetchPrimaryInviteLinks,
createOrUpdatePublicLinkActionTypes, createOrUpdatePublicLinkActionTypes,
createOrUpdatePublicLink, createOrUpdatePublicLink,
disableInviteLinkLinkActionTypes,
disableInviteLink,
}; };
View Options

web/cpp/SQLiteQueryExecutorBindings.cpp

// @flow // @flow
import type { import type {
FetchInviteLinksResponse, FetchInviteLinksResponse,
InviteLinkVerificationRequest, InviteLinkVerificationRequest,
InviteLinkVerificationResponse, InviteLinkVerificationResponse,
CreateOrUpdatePublicLinkRequest, CreateOrUpdatePublicLinkRequest,
InviteLink, InviteLink,
DisableInviteLinkRequest,
DisableInviteLinkPayload,
} from '../types/link-types.js'; } from '../types/link-types.js';
import type { CallServerEndpoint } from '../utils/call-server-endpoint.js'; import type { CallServerEndpoint } from '../utils/call-server-endpoint.js';
const verifyInviteLinkActionTypes = Object.freeze({ const verifyInviteLinkActionTypes = Object.freeze({
started: 'VERIFY_INVITE_LINK_STARTED', started: 'VERIFY_INVITE_LINK_STARTED',
success: 'VERIFY_INVITE_LINK_SUCCESS', success: 'VERIFY_INVITE_LINK_SUCCESS',
failed: 'VERIFY_INVITE_LINK_FAILED', failed: 'VERIFY_INVITE_LINK_FAILED',
}); });
▲ Show 20 Lines • Show All 53 Lines • ▼ Show 20 Lines return {
role: response.role, role: response.role,
communityID: response.communityID, communityID: response.communityID,
expirationTime: response.expirationTime, expirationTime: response.expirationTime,
limitOfUses: response.limitOfUses, limitOfUses: response.limitOfUses,
numberOfUses: response.numberOfUses, numberOfUses: response.numberOfUses,
}; };
}; };
const disableInviteLinkLinkActionTypes = Object.freeze({
started: 'DISABLE_INVITE_LINK_STARTED',
success: 'DISABLE_INVITE_LINK_SUCCESS',
failed: 'DISABLE_INVITE_LINK_FAILED',
});
const disableInviteLink =
(
callServerEndpoint: CallServerEndpoint,
): ((
request: DisableInviteLinkRequest,
) => Promise<DisableInviteLinkPayload>) =>
async request => {
await callServerEndpoint('disable_invite_link', request);
return {
name: request.name,
communityID: request.communityID,
};
};
export { export {
verifyInviteLinkActionTypes, verifyInviteLinkActionTypes,
verifyInviteLink, verifyInviteLink,
fetchPrimaryInviteLinkActionTypes, fetchPrimaryInviteLinkActionTypes,
fetchPrimaryInviteLinks, fetchPrimaryInviteLinks,
createOrUpdatePublicLinkActionTypes, createOrUpdatePublicLinkActionTypes,
createOrUpdatePublicLink, createOrUpdatePublicLink,
disableInviteLinkLinkActionTypes,
disableInviteLink,
}; };
View Options

web/shared-worker/_generated/comm_query_executor.wasm

Loading...
View Options

web/shared-worker/queries/fetch-messages-queries.test.js

Loading...
View Options

web/shared-worker/types/entities.js

Loading...
View Options

web/shared-worker/types/sqlite-query-executor.js

Loading...