HomePhabricator
Diffusion Comm 4597a9a14843

[lib] Implement `messageContentForServerDB` for `messageTypes.MULTIMEDIA`

Tags
None
Referenced Files
F175861: 8ddede.png
Sep 22 2022, 6:03 AM
File Not Attached
F175860: 20a7c2.png
Sep 22 2022, 6:03 AM
File Not Attached
Subscribers
None

Description

[lib] Implement messageContentForServerDB for messageTypes.MULTIMEDIA

Summary:
The messageContentForServerDB(...) function determines what gets put in the content column of the messages table.

The current content column for media messages is a "stringified" list of uploadIDs. However, the content column is never used after it's been stored in the DB. Rather, messages are joined with corresponding uploads via the container column of the uploads table. Therefore this change isn't "destructive."


Depends on D5195

Test Plan:
Rows in the messages and uploads column appear as expected:

20a7c2.png (54×3 px, 66 KB)

8ddede.png (194×2 px, 214 KB)

Reviewers: abosh, tomek, marcin

Reviewed By: tomek

Subscribers: ashoat

Differential Revision: https://phab.comm.dev/D5197

Details

Provenance
atulAuthored on Sep 22 2022, 5:59 AM
Reviewer
tomek
Differential Revision
D5197: [lib] Implement `messageContentForServerDB` for `messageTypes.MULTIMEDIA`
Parents
rCOMMd925ac6567b0: [native] Move tabNavigator listener to useEffect
Branches
Unknown
Tags
Unknown