Paths

Table of Contentst

[lib] remove the word reply/replies from InlineEngagement
ClosedPublic
Actions

Authored by ginsu on Jan 20 2023, 5:11 AM.

Details

Reviewers

atul
tomek
ashoat

Commits

rCOMM2adf8782038a: [lib] remove the word reply/replies from InlineEngagement

Summary

Removed the word reply/replies from InlineEngagement to reflect the new design

Screenshot 2023-01-19 at 5.50.52 PM.png (1×1 px, 139 KB)

Linear Task: ENG-2720

Test Plan

Please see the screenshots to see the before and after of my changes

Before:

Screenshot 2023-01-19 at 11.56.10 PM.png (1×3 px, 1 MB)

After:

Screenshot 2023-01-19 at 11.56.29 PM.png (1×3 px, 1 MB)

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ginsu created this revision.Jan 20 2023, 5:11 AM

Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptJan 20 2023, 5:11 AM

ginsu edited the summary of this revision. (Show Details)Jan 20 2023, 5:17 AM

ginsu edited the test plan for this revision. (Show Details)

ginsu added reviewers: atul, tomek.

ginsu attached a referenced file: F335487: Screenshot 2023-01-19 at 11.56.29 PM.png. (Show Details)

ginsu attached a referenced file: F335486: Screenshot 2023-01-19 at 11.56.10 PM.png. (Show Details)

ginsu attached a referenced file: F335485: Screenshot 2023-01-19 at 5.50.52 PM.png. (Show Details)

Harbormaster completed remote builds in B15488: Diff 21128.Jan 20 2023, 5:26 AM

ginsu requested review of this revision.Jan 20 2023, 5:26 AM

ginsu added a child revision: D6318: [web] implement new InlineEngagement design.Jan 20 2023, 5:32 AM

ashoat accepted this revision.Jan 20 2023, 2:56 PM

ashoat added inline comments.

lib/hooks/inline-engagement-text.react.js
10 ↗	(On Diff #21128)	No need for a template literal. Just cast as a string

This revision is now accepted and ready to land.Jan 20 2023, 2:56 PM

address ashoat's comments

Harbormaster completed remote builds in B15505: Diff 21150.Jan 20 2023, 6:11 PM

Closed by commit rCOMM2adf8782038a: [lib] remove the word reply/replies from InlineEngagement. · Explain WhyJan 20 2023, 7:18 PM

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMM2adf8782038a: [lib] remove the word reply/replies from InlineEngagement.

Revision Contents
Changeset List

Path

Size

lib/

hooks/

inline-engagement-text.react.js

2 lines

Diff 21150

View Options

lib/hooks/inline-engagement-text.react.js

[lib] remove the word reply/replies from InlineEngagementClosedPublicActions