Based off this comment in Linear, we want to add the word reply/replies in order to differentiate the thread replies from the reactions
Linear Task: https://linear.app/comm/issue/ENG-3103/update-inlineengagement-for-native
Details
Details
Summary
Test Plan
Please see the screenshots below:
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
Herald added subscribers: tomek, ashoat. · View Herald TranscriptJun 26 2023, 3:25 PM2023-06-26 15:25:19 (UTC-7)
ginsu edited the summary of this revision. (Show Details)Jun 26 2023, 3:28 PM2023-06-26 15:28:10 (UTC-7)
Harbormaster completed remote builds in B20512: Diff 28124.Jun 26 2023, 3:43 PM2023-06-26 15:43:16 (UTC-7)
This revision is now accepted and ready to land.Jun 27 2023, 1:17 AM2023-06-27 01:17:14 (UTC-7)
ginsu added a child revision: D8496: [native] clean up InlineEngagement component.Jul 13 2023, 10:03 AM2023-07-13 10:03:38 (UTC-7)
ginsu retitled this revision from [native] add reply/replies back to sidebar inline engagement to [lib] add reply/replies back to sidebar inline engagement.Jul 26 2023, 10:56 AM2023-07-26 10:56:03 (UTC-7)
Harbormaster completed remote builds in B21387: Diff 29299.Jul 31 2023, 12:23 PM2023-07-31 12:23:58 (UTC-7)
Closed by commit rCOMM206d67e9da1c: [lib] add reply/replies back to sidebar inline engagement (authored by ginsu). · Explain WhyJul 31 2023, 1:19 PM2023-07-31 13:19:11 (UTC-7)
This revision was automatically updated to reflect the committed changes.