Page MenuHomePhabricator
Differential D12374

[lib] factor out getting and updating device list to hook
ClosedPublic
Actions

Authored by kamil on Jun 10 2024, 4:32 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 17, 12:47 AM
Unknown Object (File)
Mon, Apr 7, 10:48 AM
Unknown Object (File)
Mon, Apr 7, 9:11 AM
Unknown Object (File)
Sun, Apr 6, 9:13 PM
Unknown Object (File)
Sun, Apr 6, 11:21 AM
Unknown Object (File)
Sun, Apr 6, 9:32 AM
Unknown Object (File)
Mar 3 2025, 6:48 AM
Unknown Object (File)
Mar 3 2025, 6:48 AM
View All 72 Files
Subscribers
ashoat

Details

Reviewers
bartek
tomek
Commits
rCOMMbe2d7d3dfc7c: [lib] factor out getting and updating device list to hook
Summary

We want to reuse this code in more places (later in the stack).

Depends on D12373

Test Plan

Test plan from D12330

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil created this revision.Jun 10 2024, 4:32 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptJun 10 2024, 4:32 AM
Harbormaster completed remote builds in B29567: Diff 41154.Jun 10 2024, 4:48 AM
kamil added a child revision: D12376: [lib] handle `IdentityDeviceListUpdated` P2P message.Jun 10 2024, 4:54 AM
kamil published this revision for review.Jun 10 2024, 9:51 AM
tomek accepted this revision.Jun 12 2024, 5:53 AM
This revision is now accepted and ready to land.Jun 12 2024, 5:53 AM
bartek accepted this revision.Jun 12 2024, 8:55 AM
Closed by commit rCOMMbe2d7d3dfc7c: [lib] factor out getting and updating device list to hook (authored by kamil). · Explain WhyJun 13 2024, 9:08 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMMbe2d7d3dfc7c: [lib] factor out getting and updating device list to hook.
kamil added a child revision: D12430: [lib] broadcast updates to all peers after Secondary Device registration success.Jun 13 2024, 9:37 AM
kamil removed a child revision: D12430: [lib] broadcast updates to all peers after Secondary Device registration success.Jun 17 2024, 4:20 AM
kamil added a child revision: D12430: [lib] broadcast updates to all peers after Secondary Device registration success.Jun 19 2024, 2:22 PM
kamil removed a child revision: D12430: [lib] broadcast updates to all peers after Secondary Device registration success.Jun 19 2024, 2:23 PM

Revision Contents
Changeset List

PathSize
lib/
handlers/
22 lines
hooks/
22 lines

Diff 41297

View Options

lib/handlers/user-infos-handler.react.js

// @flow // @flow
import { type BaseStoreOpsHandlers } from './base-ops.js'; import { type BaseStoreOpsHandlers } from './base-ops.js';
import type { RawThreadInfo } from '../types/minimally-encoded-thread-permissions-types.js'; import type { RawThreadInfo } from '../types/minimally-encoded-thread-permissions-types.js';
import type { import type {
ClientDBThreadInfo, ClientDBThreadInfo,
MinimallyEncodedRawThreadInfos, RawThreadInfos,
ThreadStore, ThreadStore,
} from '../types/thread-types.js'; } from '../types/thread-types.js';
import { import {
convertClientDBThreadInfoToRawThreadInfo, convertClientDBThreadInfoToRawThreadInfo,
convertRawThreadInfoToClientDBThreadInfo, convertRawThreadInfoToClientDBThreadInfo,
} from '../utils/thread-ops-utils.js'; } from '../utils/thread-ops-utils.js';
export type RemoveThreadOperation = { export type RemoveThreadOperation = {
Show All 24 Linesexport type ClientDBThreadStoreOperation =
| RemoveThreadOperation | RemoveThreadOperation
| RemoveAllThreadsOperation | RemoveAllThreadsOperation
| ClientDBReplaceThreadOperation; | ClientDBReplaceThreadOperation;
export const threadStoreOpsHandlers: BaseStoreOpsHandlers< export const threadStoreOpsHandlers: BaseStoreOpsHandlers<
ThreadStore, ThreadStore,
ThreadStoreOperation, ThreadStoreOperation,
ClientDBThreadStoreOperation, ClientDBThreadStoreOperation,
MinimallyEncodedRawThreadInfos, RawThreadInfos,
ClientDBThreadInfo, ClientDBThreadInfo,
> = { > = {
processStoreOperations( processStoreOperations(
store: ThreadStore, store: ThreadStore,
ops: $ReadOnlyArray<ThreadStoreOperation>, ops: $ReadOnlyArray<ThreadStoreOperation>,
): ThreadStore { ): ThreadStore {
if (ops.length === 0) { if (ops.length === 0) {
return store; return store;
▲ Show 20 Lines • Show All 43 Lines • Show Last 20 Lines
View Options

lib/hooks/peer-list-hooks.js

// @flow // @flow
import { type BaseStoreOpsHandlers } from './base-ops.js'; import { type BaseStoreOpsHandlers } from './base-ops.js';
import type { RawThreadInfo } from '../types/minimally-encoded-thread-permissions-types.js'; import type { RawThreadInfo } from '../types/minimally-encoded-thread-permissions-types.js';
import type { import type {
ClientDBThreadInfo, ClientDBThreadInfo,
MinimallyEncodedRawThreadInfos, RawThreadInfos,
ThreadStore, ThreadStore,
} from '../types/thread-types.js'; } from '../types/thread-types.js';
import { import {
convertClientDBThreadInfoToRawThreadInfo, convertClientDBThreadInfoToRawThreadInfo,
convertRawThreadInfoToClientDBThreadInfo, convertRawThreadInfoToClientDBThreadInfo,
} from '../utils/thread-ops-utils.js'; } from '../utils/thread-ops-utils.js';
export type RemoveThreadOperation = { export type RemoveThreadOperation = {
Show All 24 Linesexport type ClientDBThreadStoreOperation =
| RemoveThreadOperation | RemoveThreadOperation
| RemoveAllThreadsOperation | RemoveAllThreadsOperation
| ClientDBReplaceThreadOperation; | ClientDBReplaceThreadOperation;
export const threadStoreOpsHandlers: BaseStoreOpsHandlers< export const threadStoreOpsHandlers: BaseStoreOpsHandlers<
ThreadStore, ThreadStore,
ThreadStoreOperation, ThreadStoreOperation,
ClientDBThreadStoreOperation, ClientDBThreadStoreOperation,
MinimallyEncodedRawThreadInfos, RawThreadInfos,
ClientDBThreadInfo, ClientDBThreadInfo,
> = { > = {
processStoreOperations( processStoreOperations(
store: ThreadStore, store: ThreadStore,
ops: $ReadOnlyArray<ThreadStoreOperation>, ops: $ReadOnlyArray<ThreadStoreOperation>,
): ThreadStore { ): ThreadStore {
if (ops.length === 0) { if (ops.length === 0) {
return store; return store;
▲ Show 20 Lines • Show All 43 Lines • Show Last 20 Lines