Page MenuHomePhabricator

[lib] Add type for thick thread avatar update request
ClosedPublic

Authored by bartek on Wed, Sep 11, 1:32 AM.
Tags
None
Referenced Files
F2763351: D13284.id44166.diff
Thu, Sep 19, 10:57 AM
F2757099: D13284.id44148.diff
Thu, Sep 19, 2:23 AM
F2755849: D13284.id44165.diff
Wed, Sep 18, 10:34 PM
Unknown Object (File)
Wed, Sep 18, 3:04 PM
Unknown Object (File)
Wed, Sep 18, 6:08 AM
Unknown Object (File)
Wed, Sep 18, 3:04 AM
Unknown Object (File)
Tue, Sep 17, 8:05 PM
Unknown Object (File)
Mon, Sep 16, 8:41 PM
Subscribers

Details

Summary

Some details in this Linear comment.

The "avatar update request" requires another, keyserver-agnostic type for encrypted images. The type can be used for setting image avatars for DM threads.

Test Plan

Flow, eslint. Used later in the stack

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek held this revision as a draft.
bartek published this revision for review.Wed, Sep 11, 2:05 AM
bartek added reviewers: tomek, inka.
bartek added inline comments.
lib/types/avatar-types.js
47 ↗(On Diff #44026)

I cannot think of a good name for this.
Theoretically, this type can be reused later for user avatars when we start syncing them without keyserver, so thick_thread_image is too narrow, should be something keyserverless and general

tomek added inline comments.
lib/types/avatar-types.js
47 ↗(On Diff #44026)

Maybe non_keyserver_image?

This revision is now accepted and ready to land.Wed, Sep 11, 2:41 AM