Page MenuHomePhabricator

[lib] Replace `threadInChatList` with `useThreadInChatList` in `useWatchThread`
ClosedPublic

Authored by atul on May 9 2024, 5:58 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 26 2025, 2:21 AM
Unknown Object (File)
Jan 24 2025, 8:18 PM
Unknown Object (File)
Jan 24 2025, 5:59 AM
Unknown Object (File)
Jan 24 2025, 5:59 AM
Unknown Object (File)
Jan 24 2025, 5:59 AM
Unknown Object (File)
Jan 24 2025, 5:59 AM
Unknown Object (File)
Jan 24 2025, 5:58 AM
Unknown Object (File)
Jan 19 2025, 9:36 PM
Subscribers
None

Details

Summary

Similar to rest of stack, but for useWatchThread.


Depends on D11980

Test Plan

flow + close reading

Set log statements in useWatchThread and ensured that values before and after were identical.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul published this revision for review.May 9 2024, 6:01 PM
tomek added inline comments.
lib/shared/thread-utils.js
1299–1318 ↗(On Diff #40045)

We usually prefer to call hooks top level, without modifying their result

This revision is now accepted and ready to land.May 10 2024, 3:48 AM