Differential D13386

[lib] Disable editing failed messages
ClosedPublic
Actions

Authored by tomek on Wed, Sep 18, 10:23 AM.

Tags

None

Referenced Files

	F2766613: D13386.id44324.diff
	Thu, Sep 19, 3:44 PM

	F2764397: D13386.id.diff
	Thu, Sep 19, 12:58 PM

	F2756287: D13386.id44319.diff
	Thu, Sep 19, 12:21 AM

	Unknown Object (File)
	Wed, Sep 18, 6:11 PM

	Unknown Object (File)
	Wed, Sep 18, 6:10 PM

	Unknown Object (File)
	Wed, Sep 18, 6:10 PM

	Unknown Object (File)
	Wed, Sep 18, 4:52 PM

Subscribers

Details

Reviewers

kamil
will

Commits

rCOMMead0ca12ea9b: [lib] Disable editing failed messages

Summary

https://linear.app/comm/issue/ENG-9333/disable-editing-failed-messages

Test Plan

On both native and web, in thin and thick thread check if the edit button is visible for successful messages and isn't shown for unsuccessful.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Wed, Sep 18, 10:23 AM

Herald added a subscriber: ashoat. · View Herald TranscriptWed, Sep 18, 10:23 AM

Harbormaster completed remote builds in B31704: Diff 44319.Wed, Sep 18, 10:41 AM

tomek requested review of this revision.Wed, Sep 18, 10:41 AM

will accepted this revision.Wed, Sep 18, 12:44 PM

This revision is now accepted and ready to land.Wed, Sep 18, 12:44 PM

kamil accepted this revision.Thu, Sep 19, 1:07 AM

Closed by commit rCOMMead0ca12ea9b: [lib] Disable editing failed messages (authored by tomek). · Explain WhyThu, Sep 19, 3:07 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMMead0ca12ea9b: [lib] Disable editing failed messages.

Revision Contents
Changeset List

Path

Size

lib/

shared/

edit-messages-utils.js

2 lines

Diff 44324

lib/shared/edit-messages-utils.js

Loading...