Page MenuHomePhabricator

[native] Add delete message spec to the C++ layer
AcceptedPublic

Authored by tomek on Thu, Apr 3, 8:27 AM.
Tags
None
Referenced Files
F5297865: D14536.id.diff
Mon, Apr 7, 10:17 PM
F5287522: D14536.id47643.diff
Mon, Apr 7, 1:21 PM
F5286421: D14536.diff
Mon, Apr 7, 11:38 AM
Unknown Object (File)
Mon, Apr 7, 12:03 AM
Unknown Object (File)
Sun, Apr 6, 3:40 PM
Unknown Object (File)
Sun, Apr 6, 3:40 PM
Unknown Object (File)
Sun, Apr 6, 2:41 PM
Unknown Object (File)
Sun, Apr 6, 2:39 PM
Subscribers
None

Details

Reviewers
kamil
ashoat
Summary

Create C++ types for persisted delete messages.

https://linear.app/comm/issue/ENG-10325/create-a-new-message-type

Depends on D14535

Test Plan

Tested later in the stack by deleting a message in a thick threads, closing, and reopening the app - the message should be displayed as deleted.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

tomek requested review of this revision.Thu, Apr 3, 8:45 AM

Do we need to do anything for it to be included in the Android build, eg. update some CMakeLists.txt file?

This revision is now accepted and ready to land.Thu, Apr 3, 8:49 PM