The class component just had a single render function, so it was an easy migration.
This diff should be a clean refactor. I'll add some memoization in the next diff.
Depends on D13671
Differential D13672 Authored by ashoat on Oct 9 2024, 8:21 PM.
Details
Summary The class component just had a single render function, so it was an easy migration. This diff should be a clean refactor. I'll add some memoization in the next diff. Depends on D13671 Test Plan Flow, and playing around with web app in local environment with robotext, multimedia, and text messages, and sidebars and reactions on each
Diff Detail
Event TimelineHarbormaster completed remote builds in B32156: Diff 45030.Oct 9 2024, 8:42 PM2024-10-09 20:42:17 (UTC-7) This revision is now accepted and ready to land.Oct 10 2024, 8:04 AM2024-10-10 08:04:35 (UTC-7) Closed by commit rCOMMea1b61390ab3: [web] Convert ComposedMessage to function component (authored by ashoat). · Explain WhyOct 13 2024, 6:43 PM2024-10-13 18:43:44 (UTC-7) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 45113 web/chat/composed-message.react.js
|