Page MenuHomePhabricator
Differential D7375

[keyserver] Introduce `assignUserContainerToMedia`
AbandonedPublic
Actions

Authored by atul on Apr 10 2023, 12:11 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 18, 8:35 PM
Unknown Object (File)
Mon, Nov 18, 11:26 AM
Unknown Object (File)
Mon, Nov 18, 7:17 AM
Unknown Object (File)
Mon, Nov 18, 7:01 AM
Unknown Object (File)
Mon, Nov 18, 6:58 AM
Unknown Object (File)
Fri, Nov 8, 9:24 PM
Unknown Object (File)
Fri, Nov 8, 9:21 PM
Unknown Object (File)
Tue, Nov 5, 4:50 AM
View All 70 Files
Subscribers
tomek

Details

Reviewers
ashoat
ginsu
Summary

assignUserContainerToMedia will be used as part of updateUserAvatarResponder in subsequent diff to handle image user avatars.

Test Plan
  1. Created dummy entry in uploads table.
  2. Ran query to ensure container was correctly set.
  3. Deleted dummy entry.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul created this revision.Apr 10 2023, 12:11 PM
Herald added a subscriber: tomek. · View Herald TranscriptApr 10 2023, 12:11 PM
atul added a child revision: D7376: [keyserver] Introduce `assignThreadContainerToMedia`.Apr 10 2023, 12:12 PM
atul published this revision for review.Apr 10 2023, 12:15 PM
Harbormaster completed remote builds in B18199: Diff 24919.Apr 10 2023, 12:27 PM
ashoat requested changes to this revision.Apr 10 2023, 1:25 PM

Where is this used? Can you include the usage in this diff please? It's hard to review these standalone functions without seeing how they're used

This revision now requires changes to proceed.Apr 10 2023, 1:25 PM
atul abandoned this revision.Apr 10 2023, 1:25 PM

Revision Contents
Changeset List

PathSize
keyserver/
src/
updaters/
20 lines

Diff 24919

View Options

keyserver/src/updaters/upload-updaters.js

Loading...