Page MenuHomePhabricator
Differential D10807

[lib] Remove support for `LegacyRawThreadInfo` in `getRelativeMemberInfos`
ClosedPublic
Actions

Authored by atul on Jan 24 2024, 9:54 PM.
Tags
None
Referenced Files
F1799173: D10807.id36337.diff
Mon, May 20, 3:24 AM
F1799172: D10807.id36336.diff
Mon, May 20, 3:24 AM
F1799171: D10807.id36327.diff
Mon, May 20, 3:24 AM
F1799170: D10807.id36118.diff
Mon, May 20, 3:24 AM
F1799167: D10807.id.diff
Mon, May 20, 3:24 AM
F1799166: D10807.diff
Mon, May 20, 3:24 AM
Unknown Object (File)
Apr 4 2024, 4:30 AM
Unknown Object (File)
Mar 9 2024, 10:01 AM
View All 21 Files
Subscribers
None

Details

Reviewers
ashoat
ginsu
tomek
Commits
rCOMM74f697975705: [lib] Remove support for `LegacyRawThreadInfo` in `getRelativeMemberInfos`
Summary

Gets things to a much more readable place again.

Based this on the "before" code from D10235 (https://phab.comm.dev/file/data/m6qxcyyopjoqtkp4vuls/PHID-FILE-uwkm4w7aiiwdont5iviy/lib_selectors_user-selectors.js)

Depends on D10806

Test Plan

flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.Jan 24 2024, 9:54 PM
Harbormaster completed remote builds in B26142: Diff 36118.Jan 24 2024, 10:14 PM
atul requested review of this revision.Jan 24 2024, 10:14 PM
atul added a child revision: D10810: Remove branching on `.minimallyEncoded` where no longer relevant.Jan 24 2024, 10:19 PM
tomek accepted this revision.Jan 25 2024, 2:53 AM
tomek added inline comments.
lib/selectors/user-selectors.js
71–83 ↗(On Diff #36118)

I think we can keep relativeMemberInfo variable

This revision is now accepted and ready to land.Jan 25 2024, 2:53 AM
atul updated this revision to Diff 36327.Jan 29 2024, 3:14 PM

rebase

Harbormaster completed remote builds in B26287: Diff 36327.Jan 29 2024, 3:33 PM
atul added inline comments.Jan 29 2024, 4:53 PM
lib/selectors/user-selectors.js
71–83 ↗(On Diff #36118)

True! Will update, thanks for suggestion.

atul updated this revision to Diff 36336.Jan 29 2024, 4:53 PM

address feedback from @tomek

Harbormaster completed remote builds in B26294: Diff 36336.Jan 29 2024, 5:13 PM
Closed by commit rCOMM74f697975705: [lib] Remove support for `LegacyRawThreadInfo` in `getRelativeMemberInfos` (authored by atul). · Explain WhyJan 29 2024, 5:36 PM
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM74f697975705: [lib] Remove support for `LegacyRawThreadInfo` in `getRelativeMemberInfos`.

Revision Contents
Changeset List

PathSize
lib/
selectors/
76 lines

Diff 36337

View Options

lib/selectors/user-selectors.js

Loading...