Page MenuHomePhabricator

[native] utilize memoization in the MesssageHeader component
ClosedPublic

Authored by ginsu on Oct 6 2023, 3:54 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jun 9, 7:11 AM
Unknown Object (File)
Apr 27 2024, 4:45 PM
Unknown Object (File)
Apr 27 2024, 1:06 PM
Unknown Object (File)
Apr 27 2024, 1:06 PM
Unknown Object (File)
Apr 27 2024, 1:06 PM
Unknown Object (File)
Apr 27 2024, 1:04 PM
Unknown Object (File)
Apr 27 2024, 12:48 PM
Unknown Object (File)
Apr 25 2024, 3:04 PM
Subscribers

Details

Summary

I plan to make the user profiles accessible from the message author username in the MessageHeader. However, I noticed that MessageHeader could be better memoized. I figured it would be helpful to quickly improve the memoization here before I make this an access point for the user profile bottom sheet

Depends on D9289

Test Plan

confirmed that MessageHeader still works and looks the same

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ginsu added reviewers: atul, inka.
ginsu edited the summary of this revision. (Show Details)
ginsu requested review of this revision.Oct 6 2023, 4:11 AM
This revision is now accepted and ready to land.Oct 9 2023, 12:41 PM