[lib] Make media metadata upload to keyserver optional
ClosedPublic
Actions

Authored by bartek on Sep 11 2024, 8:21 AM.

Details

Reviewers

kamil
tomek

Commits

rCOMMd641b6e240a4: [lib] Make media metadata upload to keyserver optional

Summary

Part of ENG-8708.

Updated useBlobServiceUpload() to perform keyserver metadata upload only when keyserverOrThreadID is present.

Test Plan

Tested together with other diffs in the stack

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Sep 11 2024, 8:21 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptSep 11 2024, 8:21 AM

bartek added a child revision: D13281: [web][native] Enable blob uploads for thick threads in input-state-container.Sep 11 2024, 8:23 AM

Harbormaster completed remote builds in B31518: Diff 44022.Sep 11 2024, 8:38 AM

bartek added a child revision: D13286: [web] Support setting image avatars for thick threads.Sep 11 2024, 8:41 AM

bartek published this revision for review.Sep 11 2024, 9:01 AM

bartek added inline comments.

lib/actions/upload-actions.js
245–256 ↗	(On Diff #44022)	The logic here matches what we do on keyserver: here, here and here

bartek added a reviewer: tomek.Sep 11 2024, 9:09 AM

tomek accepted this revision.Sep 11 2024, 9:37 AM

This revision is now accepted and ready to land.Sep 11 2024, 9:37 AM

bartek mentioned this in D13294: [native] Update avatar hooks to support skipping keyserver upload.Sep 11 2024, 5:57 PM

kamil accepted this revision.Sep 12 2024, 10:17 AM

Closed by commit rCOMMd641b6e240a4: [lib] Make media metadata upload to keyserver optional. · Explain WhySep 13 2024, 12:45 PM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMd641b6e240a4: [lib] Make media metadata upload to keyserver optional.

bartek mentioned this in rCOMMdf0080acb36e: [native] Update avatar hooks to support skipping keyserver upload.

Revision Contents
Changeset List

Path

Size

lib/

actions/

upload-actions.js

47 lines

Diff 44160

View Options

lib/actions/upload-actions.js

[lib] Make media metadata upload to keyserver optionalClosedPublicActions