Refactor diff to reduce tech debt when it came to building the localID for a message.
Addresses https://linear.app/comm/issue/ENG-2852/factor-out-localid-logic-into-helper-functionhook
Depends on D8711
Differential D8807 Authored by ginsu on Aug 14 2023, 4:36 PM.
Details Summary Refactor diff to reduce tech debt when it came to building the localID for a message. Addresses https://linear.app/comm/issue/ENG-2852/factor-out-localid-logic-into-helper-functionhook Depends on D8711 Test Plan flow and confirmed that there were no regressions when sending a message
Diff Detail
Event TimelineHerald added subscribers: tomek, ashoat. · View Herald TranscriptAug 14 2023, 4:36 PM2023-08-14 16:36:05 (UTC+0) ginsu edited the summary of this revision. (Show Details)Aug 14 2023, 4:37 PM2023-08-14 16:37:44 (UTC+0) ginsu added a parent revision: D8711: [web] move reaction message action type logic into useSendReaction. Harbormaster completed remote builds in B21731: Diff 29865.Aug 14 2023, 4:56 PM2023-08-14 16:56:13 (UTC+0) This revision is now accepted and ready to land.Aug 16 2023, 9:12 AM2023-08-16 09:12:05 (UTC+0) Harbormaster completed remote builds in B22133: Diff 30369.Aug 28 2023, 3:10 AM2023-08-28 03:10:08 (UTC+0) Closed by commit rCOMMdd35f6aeead5: [lib/native/web] factor out localID logic into helper hook. · Explain WhyAug 28 2023, 3:34 AM2023-08-28 03:34:46 (UTC+0) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 30379 |