Page MenuHomePhabricator

[lib] Replace `threadHasPermission` with `useThreadHasPermission` in `useCanEditMessage`
ClosedPublic

Authored by atul on Apr 26 2024, 12:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Oct 10, 8:06 PM
Unknown Object (File)
Thu, Oct 10, 8:06 PM
Unknown Object (File)
Thu, Oct 10, 8:06 PM
Unknown Object (File)
Thu, Oct 10, 8:05 PM
Unknown Object (File)
Thu, Oct 10, 8:05 PM
Unknown Object (File)
Tue, Oct 8, 8:45 AM
Unknown Object (File)
Sep 29 2024, 4:23 PM
Unknown Object (File)
Sep 26 2024, 5:22 AM
Subscribers
None

Details

Summary

Similar to D11791, but for useCanEditMessage.


Depends on D11792

Test Plan

flow + close reading

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul requested review of this revision.Apr 26 2024, 1:07 PM
ashoat added inline comments.
lib/shared/edit-messages-utils.js
61 ↗(On Diff #39595)

I think we should do this this instead

73–77 ↗(On Diff #39595)

And then this

This revision is now accepted and ready to land.Apr 27 2024, 6:41 PM
This revision was landed with ongoing or failed builds.May 2 2024, 9:59 AM
This revision was automatically updated to reflect the committed changes.