[keyserver] Update uploads table type
ClosedPublic
Actions

Authored by tomek on Tue, Apr 30, 2:19 AM.

Details

Reviewers

inka
varun
kamil
ashoat

Commits

rCOMMf8fe718bd0aa: [keyserver] Update uploads table type

Summary

Currently user ID can be a UUID string, and we're using it as a container for avatars, so we have to update the type of container to match the one of the user ID.

https://linear.app/comm/issue/ENG-7316/unable-to-update-the-avatar#comment-5e10bd92

Test Plan

Check if it is possible to update an avatar (by changing an emoji avatar's background) and to send a message with an image.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Tue, Apr 30, 2:19 AM

Herald added a subscriber: ashoat. · View Herald TranscriptTue, Apr 30, 2:19 AM

Keep default null

Harbormaster completed remote builds in B28593: Diff 39681.Tue, Apr 30, 2:35 AM

Harbormaster completed remote builds in B28594: Diff 39682.

tomek requested review of this revision.Tue, Apr 30, 2:36 AM

ashoat accepted this revision.Tue, Apr 30, 4:58 AM

This revision is now accepted and ready to land.Tue, Apr 30, 4:58 AM

Closed by commit rCOMMf8fe718bd0aa: [keyserver] Update uploads table type (authored by tomek). · Explain WhyTue, Apr 30, 11:02 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMMf8fe718bd0aa: [keyserver] Update uploads table type.

ashoat mentioned this in rCOMM8bfd5038096c: [keyserver] Roll back D11838.Thu, May 9, 11:04 AM

Revision Contents
Changeset List

Path

Size

keyserver/

src/

database/

migration-config.js

11 lines

setup-db.js

2 lines

Diff 39708

View Options

keyserver/src/database/migration-config.js