Page MenuHomePhabricator
Differential D13133

[native] Rename `getAllMessages` method
ClosedPublic
Actions

Authored by tomek on Aug 21 2024, 10:13 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 26, 7:09 AM
Unknown Object (File)
Tue, Mar 11, 2:21 PM
Unknown Object (File)
Sat, Mar 8, 5:10 PM
Unknown Object (File)
Sat, Mar 8, 5:10 PM
Unknown Object (File)
Feb 26 2025, 2:01 AM
Unknown Object (File)
Feb 16 2025, 8:08 AM
Unknown Object (File)
Feb 11 2025, 12:12 PM
Unknown Object (File)
Jan 30 2025, 2:55 PM
View All 38 Files
Subscribers
ashoat

Details

Reviewers
kamil
inka
Commits
rCOMM45cdd7740bc6: [native] Rename `getAllMessages` method
Summary

The function changes in the next diff to return only a subset of messages from the thick threads.

https://linear.app/comm/issue/ENG-8699/store-only-non-pruned-messages-in-redux

Depends on D13106

Test Plan

Run the app on native and web and check if it opens correctly.

Diff Detail

Repository
rCOMM Comm
Branch
messages-pruning
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

tomek created this revision.Aug 21 2024, 10:13 AM
Herald added a subscriber: ashoat. · View Herald TranscriptAug 21 2024, 10:13 AM
tomek updated this revision to Diff 43560.Aug 21 2024, 10:15 AM

Rename

Harbormaster completed remote builds in B31205: Diff 43559.Aug 21 2024, 10:34 AM
Harbormaster completed remote builds in B31206: Diff 43560.Aug 21 2024, 10:40 AM
tomek requested review of this revision.Aug 21 2024, 10:40 AM
tomek added a child revision: D13137: [native] Fetch only a subset of messages from thick threads.Aug 22 2024, 4:39 AM
kamil accepted this revision.Aug 23 2024, 8:39 AM
kamil added inline comments.
native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.cpp
1487
2792
This revision is now accepted and ready to land.Aug 23 2024, 8:39 AM
tomek marked 2 inline comments as done.Aug 27 2024, 2:25 AM
tomek updated this revision to Diff 43666.Aug 27 2024, 2:25 AM

Update names

Harbormaster completed remote builds in B31274: Diff 43666.Aug 27 2024, 2:44 AM
Closed by commit rCOMM45cdd7740bc6: [native] Rename `getAllMessages` method (authored by tomek). · Explain WhyAug 27 2024, 8:02 AM
This revision was automatically updated to reflect the committed changes.
tomek added a commit: rCOMM45cdd7740bc6: [native] Rename `getAllMessages` method.

Revision Contents
Changeset List

PathSize
native/
cpp/
CommonCpp/
DatabaseManagers/
4 lines
4 lines
21 lines
NativeModules/
2 lines
6 lines
_generated/
redux/
4 lines
2 lines
schema/
2 lines
web/
cpp/
2 lines
shared-worker/
_generated/
queries/
22 lines
types/
2 lines
worker/
2 lines

Diff 43560

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

native/cpp/CommonCpp/NativeModules/CommCoreModule.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/NativeModules/CommCoreModule.cpp

Loading...
View Options

native/cpp/CommonCpp/_generated/commJSI-generated.cpp

Loading...
View Options

native/cpp/CommonCpp/_generated/commJSI.h

Loading...
View Options

native/redux/persist.js

Loading...
View Options

native/redux/unshim-utils.js

Loading...
View Options

native/schema/CommCoreModuleSchema.js

Loading...
View Options

web/cpp/SQLiteQueryExecutorBindings.cpp

Loading...
View Options

web/shared-worker/_generated/comm_query_executor.wasm

Loading...
View Options

web/shared-worker/queries/messages-and-media-queries.test.js

Loading...
View Options

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

Loading...
View Options

web/shared-worker/worker/process-operations.js

Loading...