Page MenuHomePhabricator
Differential D7830

[web] Added setting `Edit failed` state
AbandonedPublic
Actions

Authored by • kuba on May 16 2023, 4:10 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 15, 5:23 PM
Unknown Object (File)
Tue, Mar 11, 3:47 PM
Unknown Object (File)
Mon, Mar 10, 1:06 PM
Unknown Object (File)
Tue, Mar 4, 8:22 AM
Unknown Object (File)
Tue, Mar 4, 8:21 AM
Unknown Object (File)
Tue, Mar 4, 8:15 AM
Unknown Object (File)
Mon, Feb 24, 5:36 PM
Unknown Object (File)
Mon, Feb 24, 5:32 PM
View All 96 Files
Subscribers
ashoat
atul
tomek

Details

Reviewers
michal
inka
Summary

Set up displaying Edit failed notice.

Test Plan
  • Changed, when calling edit endpoint, in edit-message-utils.js messageID to the wrong one.
  • Edited the message. Checked if the Edit failed notice appears.
  • Removed wrong messageID.
  • Edited again. The message is correctly edited.
  • Opened new edit mode, checked if the edit failed doesn't appear.

Screenshot:

Screenshot 2023-05-16 at 13.11.46.png (284×926 px, 42 KB)

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Revision Contents
Changeset List

Diff 26514

View Options

web/chat/edit-message-provider.js

Loading...
View Options

web/chat/edit-text-message.react.js

Loading...