Page MenuHomePhabricator

[native] Update avatar hooks to support skipping keyserver upload
ClosedPublic

Authored by bartek on Sep 11 2024, 10:57 AM.
Tags
None
Referenced Files
F3204127: D13294.diff
Sat, Nov 9, 8:56 PM
Unknown Object (File)
Tue, Oct 22, 5:34 AM
Unknown Object (File)
Tue, Oct 22, 5:33 AM
Unknown Object (File)
Tue, Oct 22, 5:33 AM
Unknown Object (File)
Tue, Oct 15, 11:24 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

Update avatar-hooks for uploading media to support a flag whether to upload metadata to keyserver. This bases on mechanism introduced in D13280

Depends on D13286

Test Plan

This is tested together with next diffs 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.Sep 11 2024, 11:30 AM
bartek added inline comments.
native/avatars/avatar-hooks.js
430

This will be assigned basing on thread type in D13297

tomek added inline comments.
native/avatars/avatar-hooks.js
56

This isn't too readable on the call sites. How about replacing it with metadataLocation: 'keyserver' | 'blob_service'?

248

Why do we introduce this call?

This revision is now accepted and ready to land.Sep 12 2024, 12:42 AM
native/avatars/avatar-hooks.js
56

Metadata location is never blob-service. It's either keyserver or none (then it's caller responsibility to manage this)

native/avatars/avatar-hooks.js
56

Ah, ok. So maybe keyserver | none?

native/avatars/avatar-hooks.js
56

Yep, this looks much better, thanks!