[lib] Make messageID in useEditMessage mandatory
ClosedPublic
Actions

Authored by • kuba on May 23 2023, 4:27 AM.

Tags

None

Referenced Files

	F1708111: D7940.diff
	Mon, May 6, 6:36 AM

	Unknown Object (File)
	Sun, Apr 28, 5:52 PM

	Unknown Object (File)
	Wed, Apr 24, 7:13 AM

	Unknown Object (File)
	Mon, Apr 22, 1:32 PM

	Unknown Object (File)
	Mon, Apr 22, 1:32 PM

	Unknown Object (File)
	Mon, Apr 22, 1:32 PM

	Unknown Object (File)
	Mon, Apr 22, 1:29 PM

	Unknown Object (File)
	Mar 7 2024, 5:46 AM

Subscribers

Details

Reviewers

michal
inka

Commits

rCOMM8178288a239c: [lib] Make messageID in useEditMessage mandatory

Summary

Instead of invariant we can make messageID obligatory.

Test Plan

Edit the message on web,
Edit the message on iOS,

Diff Detail

Repository

rCOMM Comm

Branch

kuba/message-editing-web

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• kuba created this revision.May 23 2023, 4:27 AM

Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptMay 23 2023, 4:27 AM

• kuba added a parent revision: D7930: [web] Changed edit modal overlay opacity.May 23 2023, 4:27 AM

Harbormaster completed remote builds in B19656: Diff 26864.May 23 2023, 4:45 AM

• kuba requested review of this revision.May 23 2023, 4:45 AM

inka accepted this revision.May 23 2023, 7:32 AM

This revision is now accepted and ready to land.May 23 2023, 7:32 AM

Closed by commit rCOMM8178288a239c: [lib] Make messageID in useEditMessage mandatory (authored by kosmydel <kukubaczek@gmail.com>). · Explain WhyMay 24 2023, 4:01 AM

This revision was automatically updated to reflect the committed changes.

kosmydel <kukubaczek@gmail.com> added a commit: rCOMM8178288a239c: [lib] Make messageID in useEditMessage mandatory.

Revision Contents
Changeset List

Path

Size

lib/

shared/

edit-messages-utils.js

5 lines

Diff 26864

View Options

lib/shared/edit-messages-utils.js

[lib] Make messageID in useEditMessage mandatoryClosedPublicActions