Page MenuHomePhabricator
Differential D10139

[native][web] Remove `LegacyThreadInfo` where possible after updating `useRoleUserSurfacedPermissions`
ClosedPublic
Actions

Authored by atul on Dec 1 2023, 8:58 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 12, 7:38 PM
Unknown Object (File)
Wed, Apr 9, 11:46 AM
Unknown Object (File)
Wed, Apr 9, 6:54 AM
Unknown Object (File)
Wed, Apr 9, 3:59 AM
Unknown Object (File)
Tue, Apr 8, 3:42 PM
Unknown Object (File)
Mon, Apr 7, 11:45 PM
Unknown Object (File)
Wed, Apr 2, 7:41 PM
Unknown Object (File)
Wed, Apr 2, 3:52 AM
View All 83 Files
Subscribers
None

Details

Reviewers
ashoat
ginsu
tomek
rohan
Commits
rCOMMda210b7b0729: [native][web] Remove `LegacyThreadInfo` where possible after updating…
Summary

Simple LegacyThreadInfo removals after useRoleUserSurfacedPermissions update to support MinimallyEncodedThreadInfo.

This brings us to 79 usages of LegacyThreadInfo down from 700+.

Next diffs will involve ripping LegacyThreadInfo out of

  • native/chat
  • lib/selectors
  • etc

Depends on D10138

Test Plan

CI/flow/etc.

Diff Detail

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

Revision Contents
Changeset List

Diff 34130

View Options

native/calendar/calendar.react.js

Loading...
View Options

native/components/thread-list.react.js

Loading...
View Options

native/keyboard/keyboard-input-host.react.js

Loading...
View Options

native/roles/community-roles-screen.react.js

Loading...
View Options

web/roles/community-roles-modal.react.js

Loading...
View Options

web/roles/role-actions-menu.react.js

Loading...
View Options

web/roles/role-panel-entry.react.js

Loading...