Page MenuHomePhabricator

[keyserver] fetchMediaForThread should return string ids
ClosedPublic

Authored by michal on May 4 2023, 3:40 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 15, 6:55 PM
Unknown Object (File)
Mon, Apr 15, 1:54 AM
Unknown Object (File)
Mon, Apr 15, 12:54 AM
Unknown Object (File)
Sat, Apr 13, 7:56 AM
Unknown Object (File)
Thu, Apr 11, 6:06 AM
Unknown Object (File)
Tue, Apr 9, 6:27 AM
Unknown Object (File)
Fri, Apr 5, 4:59 AM
Unknown Object (File)
Fri, Apr 5, 4:59 AM
Subscribers

Details

Summary

There was a mismatch between flow types and actual object returned by fetchMediaForThread. Instead of returning string ids it returned numbers. This diff changes all the ids to strings before returning them. On the client side the id was only used as keys and was converted to string so there should be no changes in behaviour.

Depends on D7685

Test Plan

Check if media gallery works fine on native and web.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable