Paths

Table of Contentst

Differential D13294

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

Authored by bartek on Wed, Sep 11, 10:57 AM.

Tags

None

Referenced Files

	F2761526: D13294.id44050.diff
	Thu, Sep 19, 8:31 AM

	F2756643: D13294.id44153.diff
	Thu, Sep 19, 12:42 AM

	Unknown Object (File)
	Wed, Sep 18, 3:59 PM

	Unknown Object (File)
	Wed, Sep 18, 3:45 PM

	Unknown Object (File)
	Wed, Sep 18, 9:50 AM

	Unknown Object (File)
	Tue, Sep 17, 1:04 PM

	Unknown Object (File)
	Mon, Sep 16, 11:21 PM

	Unknown Object (File)
	Mon, Sep 16, 12:25 PM

View All 13 Files

Subscribers

Details

Reviewers

kamil
tomek

Commits

rCOMMdf0080acb36e: [native] Update avatar hooks to support skipping keyserver upload

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Wed, Sep 11, 10:57 AM

bartek held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptWed, Sep 11, 10:57 AM

bartek added a child revision: D13297: [native] Skip keyserver upload for thick thread avatars.Wed, Sep 11, 11:01 AM

Harbormaster completed remote builds in B31540: Diff 44049.Wed, Sep 11, 11:19 AM

bartek published this revision for review.Wed, Sep 11, 11:30 AM

bartek added inline comments.

native/avatars/avatar-hooks.js
430 ↗	(On Diff #44049)	This will be assigned basing on thread type in D13297

tomek accepted this revision.Thu, Sep 12, 12:42 AM

tomek added inline comments.

native/avatars/avatar-hooks.js
56 ↗	(On Diff #44049)	This isn't too readable on the call sites. How about replacing it with `metadataLocation: 'keyserver' \| 'blob_service'`?
248 ↗	(On Diff #44049)	Why do we introduce this call?

This revision is now accepted and ready to land.Thu, Sep 12, 12:42 AM

bartek added inline comments.Fri, Sep 13, 3:01 AM

native/avatars/avatar-hooks.js
56 ↗	(On Diff #44049)	Metadata location is never blob-service. It's either keyserver or none (then it's caller responsibility to manage this)

tomek added inline comments.Fri, Sep 13, 3:24 AM

native/avatars/avatar-hooks.js
56 ↗	(On Diff #44049)	Ah, ok. So maybe `keyserver \| none`?

Apply suggestion

bartek added inline comments.Fri, Sep 13, 4:38 AM

native/avatars/avatar-hooks.js
56 ↗	(On Diff #44049)	Yep, this looks much better, thanks!

Harbormaster failed remote builds in B31605: Diff 44150!Fri, Sep 13, 4:46 AM

Fix flow

Harbormaster completed remote builds in B31607: Diff 44152.Fri, Sep 13, 5:21 AM

Closed by commit rCOMMdf0080acb36e: [native] Update avatar hooks to support skipping keyserver upload (authored by bartek). · Explain WhyFri, Sep 13, 5:45 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMdf0080acb36e: [native] Update avatar hooks to support skipping keyserver upload.

Revision Contents
Changeset List

Path

Size

native/

account/

registration/

registration-server-call.js

5 lines

avatars/

avatar-hooks.js

156 lines

Diff 44167

native/account/registration/registration-server-call.js

Loading...

native/avatars/avatar-hooks.js

Loading...