Page MenuHomePhabricator

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

Authored by bartek on Sep 11 2024, 1:32 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 4, 5:41 PM
Unknown Object (File)
Tue, Oct 22, 5:34 AM
Unknown Object (File)
Tue, Oct 22, 5:33 AM
Unknown Object (File)
Wed, Oct 16, 4:21 AM
Unknown Object (File)
Tue, Oct 15, 12:04 PM
Unknown Object (File)
Sat, Oct 12, 1:05 AM
Unknown Object (File)
Sat, Oct 12, 1:05 AM
Unknown Object (File)
Sat, Oct 12, 1:03 AM
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.Sep 11 2024, 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.Sep 11 2024, 2:41 AM