HomePhabricator
Diffusion Comm 9f41d12aef51

[web] Replace `threadHasPermission` with `useThreadHasPermission` in…

Tags
None
Referenced Files
F1656078: 8426a2.png
Apr 24 2024, 6:58 PM
File Not Attached
F1656062: c1cee6.png
Apr 24 2024, 6:58 PM
File Not Attached
Subscribers
None

Description

[web] Replace threadHasPermission with useThreadHasPermission in EditThreadAvatar

Summary:
Followup to D11746 where we're updating client to use hook.


Depends on D11746

Test Plan:
flow/CI/etc.

Thread where I shouldn't have EDIT_THREAD_AVATAR permission doesn't let me change avatar:

c1cee6.png (414×472 px, 60 KB)

Thread where I should have EDIT_THREAD_AVATAR permission DOES let me change avatar:

8426a2.png (660×1 px, 78 KB)

Reviewers: ginsu, tomek, varun, will

Reviewed By: tomek

Subscribers: ashoat

Differential Revision: https://phab.comm.dev/D11747

Details

Provenance
atulAuthored on Apr 23 2024, 12:40 PM
Reviewer
tomek
Differential Revision
D11747: [web] Replace `threadHasPermission` with `useThreadHasPermission` in `EditThreadAvatar`
Parents
rCOMM2e193ebaa76c: [lib] Introduce `useThreadHasPermission` hook
Branches
Unknown
Tags
Unknown