Page MenuHomePhabricator
Differential D10731

[lib] Expand `RoleInfo` to `LegacyRoleInfo | MinimallyEncodedRoleInfo`
ClosedPublic
Actions

Authored by atul on Jan 18 2024, 2:10 PM.
Tags
None
Referenced Files
F1799416: D10731.id36318.diff
Mon, May 20, 3:55 AM
F1799413: D10731.id36302.diff
Mon, May 20, 3:55 AM
F1799406: D10731.id36080.diff
Mon, May 20, 3:55 AM
F1799399: D10731.id36310.diff
Mon, May 20, 3:55 AM
F1799393: D10731.id35856.diff
Mon, May 20, 3:55 AM
F1799372: D10731.id.diff
Mon, May 20, 3:55 AM
F1799370: D10731.diff
Mon, May 20, 3:55 AM
Unknown Object (File)
Mar 13 2024, 2:25 AM
View All 27 Files
Subscribers
None

Details

Reviewers
ashoat
ginsu
tomek
Commits
rCOMM4a14f1cb044c: [lib] Expand `RoleInfo` to `LegacyRoleInfo | MinimallyEncodedRoleInfo`
Summary

Part of https://linear.app/comm/issue/ENG-6366/rename-minimallyencodedrawthreadinfo-to-rawthreadinfo

Depends on D10730

Test Plan

flow, done automatically with IDE "Inline alias.." refactor

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.Jan 18 2024, 2:10 PM
atul added a child revision: D10733: [lib] Remove `LegacyRoleInfo` where possible in `lib`.Jan 18 2024, 2:14 PM
Harbormaster completed remote builds in B25934: Diff 35856.Jan 18 2024, 2:28 PM
atul requested review of this revision.Jan 18 2024, 2:28 PM
ashoat accepted this revision.Jan 21 2024, 9:51 PM
ashoat added inline comments.
lib/shared/thread-utils.js
64 ↗(On Diff #35856)

Please revert import ordering changes before landing

This revision is now accepted and ready to land.Jan 21 2024, 9:51 PM
atul updated this revision to Diff 36080.Jan 23 2024, 2:37 PM

rebase

Harbormaster completed remote builds in B26109: Diff 36080.Jan 23 2024, 3:06 PM
atul updated this revision to Diff 36302.Jan 29 2024, 1:00 PM

rebase

atul updated this revision to Diff 36310.Jan 29 2024, 1:06 PM

fix import

Harbormaster completed remote builds in B26269: Diff 36302.Jan 29 2024, 1:19 PM
Harbormaster completed remote builds in B26277: Diff 36310.Jan 29 2024, 1:35 PM
Closed by commit rCOMM4a14f1cb044c: [lib] Expand `RoleInfo` to `LegacyRoleInfo | MinimallyEncodedRoleInfo` (authored by atul). · Explain WhyJan 29 2024, 3:09 PM
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM4a14f1cb044c: [lib] Expand `RoleInfo` to `LegacyRoleInfo | MinimallyEncodedRoleInfo`.

Revision Contents
Changeset List

PathSize
lib/
shared/
6 lines
types/
3 lines
utils/
10 lines

Diff 36318

View Options

lib/shared/thread-utils.js

Loading...
View Options

lib/types/thread-types.js

Loading...
View Options

lib/utils/role-utils.js

Loading...