Differential D12872

[lib] DMOperationSpec for remove members operation
ClosedPublic
Actions

Authored by tomek on Jul 24 2024, 10:43 AM.

Tags

None

Referenced Files

	F3140314: D12872.diff
	Sun, Nov 3, 4:25 AM

	Unknown Object (File)
	Mon, Oct 28, 9:54 AM

	Unknown Object (File)
	Sat, Oct 26, 12:41 AM

	Unknown Object (File)
	Tue, Oct 22, 1:17 PM

	Unknown Object (File)
	Tue, Oct 22, 8:34 AM

	Unknown Object (File)
	Oct 2 2024, 10:14 PM

	Unknown Object (File)
	Sep 25 2024, 5:15 PM

	Unknown Object (File)
	Sep 25 2024, 5:15 PM

View All 90 Files

Subscribers

Details

Reviewers

kamil
inka

Commits

rCOMMfc12922a4b56: [lib] DMOperationSpec for remove members operation

Summary

Create the spec.

https://linear.app/comm/issue/ENG-8738/create-dmoperations-for-removemembersmessageinfo

Depends on D12871

Test Plan

Just the Flow. Further testing will be performed after it is connected with the rest of the app.

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

tomek created this revision.Jul 24 2024, 10:43 AM

Herald added a subscriber: ashoat. · View Herald TranscriptJul 24 2024, 10:43 AM

Harbormaster returned this revision to the author for changes because remote builds failed.Jul 24 2024, 11:00 AM

Harbormaster failed remote builds in B30648: Diff 42744!

tomek added a child revision: D12881: [lib] DMOperationSpec for change thread settings operation.Jul 25 2024, 8:47 AM

Rebase

Harbormaster returned this revision to the author for changes because remote builds failed.Jul 25 2024, 9:06 AM

Harbormaster failed remote builds in B30671: Diff 42775!

Rebase

Harbormaster completed remote builds in B30691: Diff 42807.Jul 26 2024, 3:26 AM

tomek requested review of this revision.Jul 26 2024, 3:26 AM

Rename

Harbormaster completed remote builds in B30705: Diff 42822.Jul 26 2024, 5:04 AM

inka added inline comments.Jul 26 2024, 5:31 AM

lib/shared/dm-ops/remove-members-spec.js
48–53 ↗	(On Diff #42822)	Is this correct for sidebars?
67 ↗	(On Diff #42822)	Should we update the unread status? Should we maybe think of doing that in the reducer instead of creating an action in all specs?

Correctly handle sidebars

Harbormaster completed remote builds in B30757: Diff 42899.Jul 29 2024, 7:02 AM

tomek added inline comments.Jul 29 2024, 7:14 AM

lib/shared/dm-ops/remove-members-spec.js
67 ↗	(On Diff #42822)	Hmmm... I guess we could update the unread status every time a new message appears in the store.

inka accepted this revision.Jul 31 2024, 2:11 AM

inka added inline comments.

lib/shared/dm-ops/remove-members-spec.js
67 ↗	(On Diff #42822)	This would also make it easier to only update the status when a message with a bigger timestamp than all previous messages appears, if we decided we wanted to do that.

This revision is now accepted and ready to land.Jul 31 2024, 2:11 AM

tomek added inline comments.Aug 1 2024, 5:17 AM

lib/shared/dm-ops/remove-members-spec.js
67 ↗	(On Diff #42822)	It seems like updating the read status and replies count is really similar. But I'm not sure how to do it well. Going to search for a good solution.

Closed by commit rCOMMfc12922a4b56: [lib] DMOperationSpec for remove members operation (authored by tomek). · Explain WhyAug 2 2024, 8:38 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMMfc12922a4b56: [lib] DMOperationSpec for remove members operation.

Revision Contents
Changeset List

Path

Size

lib/

shared/

dm-ops/

2 lines

remove-members-spec.js

90 lines

types/

23 lines

Diff 42899

lib/shared/dm-ops/dm-op-specs.js

Loading...

lib/shared/dm-ops/remove-members-spec.js

Loading...

lib/types/dm-ops.js

Loading...