[web] Remove duplicate containsInlineEngagement property
ClosedPublic
Actions

Authored by ashoat on Oct 9 2024, 8:20 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Mar 15, 3:17 AM

	Unknown Object (File)
	Feb 17 2025, 11:17 AM

	Unknown Object (File)
	Feb 17 2025, 11:17 AM

	Unknown Object (File)
	Feb 17 2025, 11:17 AM

	Unknown Object (File)
	Feb 17 2025, 11:17 AM

	Unknown Object (File)
	Feb 16 2025, 10:45 AM

	Unknown Object (File)
	Jan 31 2025, 9:13 AM

	Unknown Object (File)
	Jan 31 2025, 9:13 AM

View All 45 Files

Subscribers

None

Details

Reviewers

tomek

Commits

rCOMM1c976c657914: [web] Remove duplicate containsInlineEngagement property

Summary

The check is duplicated in the only place this property is used. We can just remove the property.

Depends on D13670

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ashoat created this revision.Oct 9 2024, 8:20 PM

ashoat added a child revision: D13672: [web] Convert ComposedMessage to function component.

Harbormaster completed remote builds in B32155: Diff 45029.Oct 9 2024, 8:37 PM

ashoat requested review of this revision.Oct 9 2024, 8:37 PM

tomek accepted this revision.Oct 10 2024, 6:58 AM

This revision is now accepted and ready to land.Oct 10 2024, 6:58 AM

Closed by commit rCOMM1c976c657914: [web] Remove duplicate containsInlineEngagement property (authored by ashoat). · Explain WhyOct 13 2024, 6:43 PM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM1c976c657914: [web] Remove duplicate containsInlineEngagement property.

Revision Contents
Changeset List

Path

Size

web/

chat/

composed-message.react.js

5 lines

Diff 45112

View Options

web/chat/composed-message.react.js

[web] Remove duplicate containsInlineEngagement propertyClosedPublicActions