[lib] Simplify `threadMemberHasPermission()`
ClosedPublic
Actions

Authored by atul on Tue, Apr 23, 10:46 AM.

Details

Reviewers

ginsu
tomek
varun
will

Commits

rCOMM24c5c5fc270d: [lib] Simplify `threadMemberHasPermission()`

Summary

This function is only consumed by appendUserInfo which is only consumed in usePotentialMemberItems. This hook is only used on client, so there's no need to support Server* or Legacy* types.

We can also skip .minimallyEncoded branch since all clients work exclusively with MinimallyEncoded* types at this point.

Depends on D11740

Test Plan

NA, flow and read through callsites.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Tue, Apr 23, 10:46 AM

Herald added a subscriber: ashoat. · View Herald TranscriptTue, Apr 23, 10:46 AM

atul added a child revision: D11742: [lib] Pull `filterOutDisabledPermissions(permissionsBitmask)` out of `getCurrentUser`.Tue, Apr 23, 10:55 AM

Harbormaster completed remote builds in B28412: Diff 39417.Tue, Apr 23, 11:09 AM

atul requested review of this revision.Tue, Apr 23, 11:09 AM

tomek accepted this revision.Wed, Apr 24, 4:25 AM

This revision is now accepted and ready to land.Wed, Apr 24, 4:25 AM

land trivial

This revision was landed with ongoing or failed builds.Wed, Apr 24, 12:23 PM

Closed by commit rCOMM24c5c5fc270d: [lib] Simplify `threadMemberHasPermission()` (authored by atul). · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM24c5c5fc270d: [lib] Simplify `threadMemberHasPermission()`.

Harbormaster completed remote builds in B28435: Diff 39443.Wed, Apr 24, 12:31 PM

Revision Contents
Changeset List

Path

Size

lib/

shared/

thread-utils.js

11 lines

Diff 39444

View Options

lib/shared/thread-utils.js

[lib] Simplify `threadMemberHasPermission()`ClosedPublicActions