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
F2770532: D13284.diff
Thu, Sep 19, 8:47 PM
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
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
No Lint Coverage
Unit
No Test Coverage

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