Page MenuHomePhabricator
Differential D13192

[lib] implement changing DM threads settings
ClosedPublic
Actions

Authored by kamil on Aug 28 2024, 5:57 AM.
Tags
None
Referenced Files
F5255609: D13192.diff
Sun, Apr 6, 8:31 AM
Unknown Object (File)
Mar 6 2025, 8:54 PM
Unknown Object (File)
Mar 6 2025, 7:12 AM
Unknown Object (File)
Mar 6 2025, 5:37 AM
Unknown Object (File)
Mar 6 2025, 4:52 AM
Unknown Object (File)
Mar 4 2025, 11:34 AM
Unknown Object (File)
Mar 4 2025, 7:16 AM
Unknown Object (File)
Mar 4 2025, 7:16 AM
View All 94 Files
Subscribers
ashoat

Details

Reviewers
tomek
inka
Commits
rCOMM0894c77808ea: [lib] implement changing DM threads settings
Summary

ENG-8979;

Because we can just return empty ChangeThreadSettingsPayload we were able to update this here, not in each component. This simplified things a lot, implicitly handled loading status, and saved a lot of other work.

To fully support avatars I created ENG-9102.

Test Plan

Test:

  1. Changing thread name
  2. Changing thread description
  3. Changing thread colors
  4. Changing thread avatars (only emojis)

Diff Detail

Repository
rCOMM Comm
Branch
logic-4
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil created this revision.Aug 28 2024, 5:57 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptAug 28 2024, 5:57 AM
Harbormaster completed remote builds in B31329: Diff 43737.Aug 28 2024, 6:14 AM
kamil updated this revision to Diff 43749.Aug 28 2024, 7:28 AM

update

kamil edited the summary of this revision. (Show Details)Aug 28 2024, 7:33 AM
kamil edited the test plan for this revision. (Show Details)
kamil added a parent revision: D13193: [lib] make it possible to send DM ops to all thread members.
kamil mentioned this in D13195: [lib] remove `newMemberIDs` from `DMThreadSettingsChanges`.Aug 28 2024, 7:42 AM
kamil published this revision for review.Aug 28 2024, 7:43 AM
kamil added a child revision: D13195: [lib] remove `newMemberIDs` from `DMThreadSettingsChanges`.
Harbormaster completed remote builds in B31339: Diff 43749.Aug 28 2024, 7:45 AM
tomek accepted this revision.Aug 28 2024, 9:13 AM
tomek added inline comments.
web/modals/threads/settings/thread-settings-utils.js
87

Why do we use null here?

This revision is now accepted and ready to land.Aug 28 2024, 9:13 AM
kamil added inline comments.Aug 29 2024, 12:18 AM
web/modals/threads/settings/thread-settings-utils.js
87

removed

kamil updated this revision to Diff 43777.Aug 29 2024, 12:19 AM

address review

Harbormaster completed remote builds in B31357: Diff 43777.Aug 29 2024, 12:36 AM
Closed by commit rCOMM0894c77808ea: [lib] implement changing DM threads settings (authored by kamil). · Explain WhyAug 29 2024, 1:16 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM0894c77808ea: [lib] implement changing DM threads settings.

Revision Contents
Changeset List

PathSize
lib/
actions/
92 lines
components/
6 lines
native/
chat/
settings/
4 lines
2 lines
2 lines
web/
modals/
threads/
settings/
4 lines

Diff 43749

View Options

lib/actions/thread-actions.js

// @flow // @flow
import type { NotificationsCreationData } from './notif-types.js';
import type { StoreOperations } from './store-ops-types.js'; import type { StoreOperations } from './store-ops-types.js';
export type MessageSourceMetadata = { export type MessageSourceMetadata = {
+messageID: string, +messageID: string,
+senderDeviceID: string, +senderDeviceID: string,
}; };
export type DBOpsEntry = export type DBOpsEntry =
| { | {
+messageSourceMetadata: MessageSourceMetadata, +messageSourceMetadata: MessageSourceMetadata,
+ops?: ?StoreOperations, +ops?: ?StoreOperations,
} }
| { | {
+ops: StoreOperations, +ops: StoreOperations,
+dmOpID?: string, +dmOpID?: string,
}
| {
+notificationsCreationData?: NotificationsCreationData,
}; };
export type DBOpsStore = { export type DBOpsStore = {
+queuedOps: $ReadOnlyArray<DBOpsEntry>, +queuedOps: $ReadOnlyArray<DBOpsEntry>,
}; };
View Options

lib/components/base-edit-thread-avatar-provider.react.js

// @flow // @flow
import type { NotificationsCreationData } from './notif-types.js';
import type { StoreOperations } from './store-ops-types.js'; import type { StoreOperations } from './store-ops-types.js';
export type MessageSourceMetadata = { export type MessageSourceMetadata = {
+messageID: string, +messageID: string,
+senderDeviceID: string, +senderDeviceID: string,
}; };
export type DBOpsEntry = export type DBOpsEntry =
| { | {
+messageSourceMetadata: MessageSourceMetadata, +messageSourceMetadata: MessageSourceMetadata,
+ops?: ?StoreOperations, +ops?: ?StoreOperations,
} }
| { | {
+ops: StoreOperations, +ops: StoreOperations,
+dmOpID?: string, +dmOpID?: string,
}
| {
+notificationsCreationData?: NotificationsCreationData,
}; };
export type DBOpsStore = { export type DBOpsStore = {
+queuedOps: $ReadOnlyArray<DBOpsEntry>, +queuedOps: $ReadOnlyArray<DBOpsEntry>,
}; };
View Options

native/chat/settings/color-selector-modal.react.js

// @flow // @flow
import type { NotificationsCreationData } from './notif-types.js';
import type { StoreOperations } from './store-ops-types.js'; import type { StoreOperations } from './store-ops-types.js';
export type MessageSourceMetadata = { export type MessageSourceMetadata = {
+messageID: string, +messageID: string,
+senderDeviceID: string, +senderDeviceID: string,
}; };
export type DBOpsEntry = export type DBOpsEntry =
| { | {
+messageSourceMetadata: MessageSourceMetadata, +messageSourceMetadata: MessageSourceMetadata,
+ops?: ?StoreOperations, +ops?: ?StoreOperations,
} }
| { | {
+ops: StoreOperations, +ops: StoreOperations,
+dmOpID?: string, +dmOpID?: string,
}
| {
+notificationsCreationData?: NotificationsCreationData,
}; };
export type DBOpsStore = { export type DBOpsStore = {
+queuedOps: $ReadOnlyArray<DBOpsEntry>, +queuedOps: $ReadOnlyArray<DBOpsEntry>,
}; };
View Options

native/chat/settings/thread-settings-description.react.js

// @flow // @flow
import type { NotificationsCreationData } from './notif-types.js';
import type { StoreOperations } from './store-ops-types.js'; import type { StoreOperations } from './store-ops-types.js';
export type MessageSourceMetadata = { export type MessageSourceMetadata = {
+messageID: string, +messageID: string,
+senderDeviceID: string, +senderDeviceID: string,
}; };
export type DBOpsEntry = export type DBOpsEntry =
| { | {
+messageSourceMetadata: MessageSourceMetadata, +messageSourceMetadata: MessageSourceMetadata,
+ops?: ?StoreOperations, +ops?: ?StoreOperations,
} }
| { | {
+ops: StoreOperations, +ops: StoreOperations,
+dmOpID?: string, +dmOpID?: string,
}
| {
+notificationsCreationData?: NotificationsCreationData,
}; };
export type DBOpsStore = { export type DBOpsStore = {
+queuedOps: $ReadOnlyArray<DBOpsEntry>, +queuedOps: $ReadOnlyArray<DBOpsEntry>,
}; };
View Options

native/chat/settings/thread-settings-name.react.js

// @flow // @flow
import type { NotificationsCreationData } from './notif-types.js';
import type { StoreOperations } from './store-ops-types.js'; import type { StoreOperations } from './store-ops-types.js';
export type MessageSourceMetadata = { export type MessageSourceMetadata = {
+messageID: string, +messageID: string,
+senderDeviceID: string, +senderDeviceID: string,
}; };
export type DBOpsEntry = export type DBOpsEntry =
| { | {
+messageSourceMetadata: MessageSourceMetadata, +messageSourceMetadata: MessageSourceMetadata,
+ops?: ?StoreOperations, +ops?: ?StoreOperations,
} }
| { | {
+ops: StoreOperations, +ops: StoreOperations,
+dmOpID?: string, +dmOpID?: string,
}
| {
+notificationsCreationData?: NotificationsCreationData,
}; };
export type DBOpsStore = { export type DBOpsStore = {
+queuedOps: $ReadOnlyArray<DBOpsEntry>, +queuedOps: $ReadOnlyArray<DBOpsEntry>,
}; };
View Options

web/modals/threads/settings/thread-settings-utils.js

Loading...