Paths

Table of Contentst

Differential D12967

[lib] Remove community from thick threads
ClosedPublic
Actions

Authored by tomek on Aug 5 2024, 3:24 AM.

Tags

None

Referenced Files

	F2763004: D12967.id43168.diff
	Thu, Sep 19, 10:36 AM

	F2754980: D12967.diff
	Wed, Sep 18, 9:16 PM

	Unknown Object (File)
	Thu, Sep 5, 4:42 PM

	Unknown Object (File)
	Thu, Sep 5, 4:42 PM

	Unknown Object (File)
	Thu, Sep 5, 4:42 PM

	Unknown Object (File)
	Thu, Sep 5, 4:42 PM

	Unknown Object (File)
	Sat, Aug 31, 4:56 PM

	Unknown Object (File)
	Fri, Aug 30, 8:32 PM

View All 35 Files

Subscribers

Details

Reviewers

kamil
inka

Commits

rCOMMdf3698446e2f: [lib] Remove community from thick threads

Summary

Thick threads can't have a community.

https://linear.app/comm/issue/ENG-8937/remove-community-from-mutablethickrawthreadinfo

More context D12914.

Test Plan

Made sure that it's possible to create a sidebar. Also checked if persisting and reading thin and thick threads from SQLite works on web.

Diff Detail

Repository

Branch

dmops

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

tomek created this revision.Aug 5 2024, 3:24 AM

Herald added a subscriber: ashoat. · View Herald TranscriptAug 5 2024, 3:24 AM

tomek added inline comments.Aug 5 2024, 3:27 AM

lib/utils/thread-ops-utils.js
47	This is for Flow. `ClientDBThreadInfo` has `community` as a mandatory nullable property, but thick threads don't have it at all.
native/redux/persist.js
405–427	We shouldn't expect any thick thread to be in the store while performing this old migration.

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 5 2024, 3:45 AM

Harbormaster failed remote builds in B30887: Diff 43067!

tomek added a child revision: D12969: [lib] Rename editor to joiner in join thread operation.Aug 5 2024, 3:53 AM

Harbormaster completed remote builds in B30887: Diff 43067.Aug 5 2024, 4:17 AM

tomek requested review of this revision.Aug 5 2024, 4:17 AM

tomek edited the summary of this revision. (Show Details)Aug 5 2024, 6:20 AM

tomek edited the summary of this revision. (Show Details)

inka accepted this revision.Aug 5 2024, 6:23 AM

This revision is now accepted and ready to land.Aug 5 2024, 6:23 AM

Closed by commit rCOMMdf3698446e2f: [lib] Remove community from thick threads (authored by tomek). · Explain WhyAug 6 2024, 3:35 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMMdf3698446e2f: [lib] Remove community from thick threads.

Revision Contents
Changeset List

Path

Size

lib/

shared/

dm-ops/

create-thread-spec.js

2 lines

thread-utils.js

1 line

types/

2 lines

thread-types.js

1 line

utils/

thread-ops-utils.js

2 lines

native/

redux/

10 lines

Diff 43067

lib/shared/dm-ops/create-thread-spec.js

Loading...

lib/shared/thread-utils.js

Loading...

lib/types/dm-ops.js

Loading...

lib/types/thread-types.js

Loading...

lib/utils/thread-ops-utils.js

Loading...

native/redux/persist.js

Loading...