[lib] Update `useRoleUserSurfacedPermissions` to handle `MinimallyEncodedThreadInfo`
ClosedPublic
Actions

Authored by atul on Dec 1 2023, 4:45 PM.

Details

Reviewers

ashoat
ginsu
tomek
rohan

Commits

rCOMM345bd8382ddf: [lib] Update `useRoleUserSurfacedPermissions` to handle…

Summary

Updated this permissions-related hook to handle MinimallyEncodedThreadInfo. This will allow subsequent removal of LegacyThreadInfo annotations, but will make those changes in separate diff.

Depends on D10137

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, 4:45 PM

atul published this revision for review.Dec 1 2023, 4:45 PM

atul added a child revision: D10139: [native][web] Remove `LegacyThreadInfo` where possible after updating `useRoleUserSurfacedPermissions`.Dec 1 2023, 4:58 PM

Harbormaster failed remote builds in B24697: Diff 34129!Dec 1 2023, 5:11 PM

ashoat accepted this revision.Dec 1 2023, 5:16 PM

This revision is now accepted and ready to land.Dec 1 2023, 5:16 PM

Harbormaster completed remote builds in B24697: Diff 34129.Dec 1 2023, 5:29 PM

Closed by commit rCOMM345bd8382ddf: [lib] Update `useRoleUserSurfacedPermissions` to handle…. · Explain WhyDec 1 2023, 8:44 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM345bd8382ddf: [lib] Update `useRoleUserSurfacedPermissions` to handle….

Revision Contents
Changeset List

Path

Size

lib/

shared/

thread-utils.js

9 lines

Diff 34129

View Options

lib/shared/thread-utils.js

[lib] Update `useRoleUserSurfacedPermissions` to handle `MinimallyEncodedThreadInfo`ClosedPublicActions