Page MenuHomePhabricator
Differential D10142

Rip `LegacyThreadInfo` out of selectors and related code
ClosedPublic
Actions

Authored by atul on Dec 1 2023, 11:24 AM.
Tags
None
Referenced Files
F5374522: D10142.id34136.diff
Wed, Apr 9, 4:33 PM
F5360312: D10142.diff
Wed, Apr 9, 12:13 PM
F5353163: D10142.id34135.diff
Wed, Apr 9, 9:51 AM
F5346206: D10142.id34143.diff
Wed, Apr 9, 3:58 AM
Unknown Object (File)
Wed, Apr 9, 12:39 AM
Unknown Object (File)
Wed, Apr 9, 12:06 AM
Unknown Object (File)
Tue, Apr 8, 10:15 PM
Unknown Object (File)
Tue, Apr 8, 9:05 PM
View All Files
Subscribers
None

Details

Reviewers
ashoat
ginsu
tomek
rohan
Commits
rCOMMaca9f25487d3: Rip `LegacyThreadInfo` out of selectors and related code
Summary

Ripped LegacyThreadInfo out of selectors and updated related code to use ThreadInfo as well.

We're now down to 6 usages of LegacyThreadInfo from ~750

WARNING: There was one instance where we had to add noop branching to web/modals/threads/settings/thread-settings-modal.react.js:68-72 to appease flow.

Depends on D10140

Test Plan

CI/flow/etc.

Diff Detail

Repository
rCOMM Comm
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul created this revision.Dec 1 2023, 11:24 AM
atul added inline comments.Dec 1 2023, 11:25 AM
web/chat/thread-list-provider.js
152 ↗(On Diff #34135)

This was accidental, it will be removed.

web/modals/threads/settings/thread-settings-modal.react.js
68–72 ↗(On Diff #34135)
NOTE: This is the one place we had to add branching to appease flow.
atul updated this revision to Diff 34136.Dec 1 2023, 11:27 AM

update

atul published this revision for review.Dec 1 2023, 11:28 AM
atul edited the summary of this revision. (Show Details)
Harbormaster completed remote builds in B24702: Diff 34135.Dec 1 2023, 11:52 AM
Harbormaster completed remote builds in B24703: Diff 34136.Dec 1 2023, 12:02 PM
ashoat accepted this revision.Dec 1 2023, 12:18 PM
This revision is now accepted and ready to land.Dec 1 2023, 12:18 PM
atul updated this revision to Diff 34143.Dec 1 2023, 12:21 PM

include more

atul edited the summary of this revision. (Show Details)Dec 1 2023, 12:21 PM
This revision was landed with ongoing or failed builds.Dec 1 2023, 12:44 PM
Closed by commit rCOMMaca9f25487d3: Rip `LegacyThreadInfo` out of selectors and related code (authored by atul). · Explain Why
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMMaca9f25487d3: Rip `LegacyThreadInfo` out of selectors and related code.
Harbormaster completed remote builds in B24710: Diff 34143.Dec 1 2023, 1:23 PM

Revision Contents
Changeset List

PathSize
lib/
selectors/
19 lines
37 lines
shared/
7 lines
8 lines
10 lines
6 lines
types/
4 lines
utils/
11 lines
native/
chat/
settings/
3 lines
push/
4 lines
selectors/
6 lines
web/
chat/
1 line
modals/
threads/
settings/
13 lines
selectors/
8 lines

Diff 34136

View Options

lib/selectors/chat-selectors.js

Loading...
View Options

lib/selectors/thread-selectors.js

Loading...
View Options

lib/shared/ancestor-threads.js

Loading...
View Options

lib/shared/message-utils.js

Loading...
View Options

lib/shared/notif-utils.js

Loading...
View Options

lib/shared/thread-utils.js

Loading...
View Options

lib/types/thread-types.js

Loading...
View Options

lib/utils/entity-helpers.js

Loading...
View Options

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

Loading...
View Options

native/push/push-handler.react.js

Loading...
View Options

native/selectors/calendar-selectors.js

Loading...
View Options

web/chat/thread-list-provider.js

Loading...
View Options

web/modals/threads/settings/thread-settings-modal.react.js

Loading...
View Options

web/selectors/chat-selectors.js

Loading...