HomePhabricator
Diffusion Comm ca407f743999

[lib] Introduce `getUploadIDsFromMediaMessageServerDBContents(...)`

Description

[lib] Introduce getUploadIDsFromMediaMessageServerDBContents(...)

Summary:
Introduce getUploadIDsFromMediaMessageServerDBContents(...) and consume in fetchUploadsForMessage(...) and assignMessageContainerToMedia(...).

Simple refactor to reduce redundant code.

Addresses feedback from @ashoat here: https://phab.comm.dev/D5087#inline-33653

Test Plan: Simple refactor, things continue to work as expected when sending media messages. Set breakpoints and ensured that values at each step were as expected.

Reviewers: abosh, tomek, marcin, varun

Reviewed By: tomek

Subscribers: ashoat

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

Details

Provenance
atulAuthored on Sep 21 2022, 12:50 PM
Reviewer
tomek
Differential Revision
D5190: [lib] Introduce `getUploadIDsFromMediaMessageServerDBContents(...)`
Parents
rCOMMf467714198b3: [CI] Add `--network-timeout 180000` to `yarn` commands
Branches
Unknown
Tags
Unknown