Paths

Table of Contentst

[keyserver] Drop the default_role column from threads
ClosedPublic
Actions

Authored by rohan on Nov 24 2023, 10:34 AM.

Details

Reviewers

atul
ginsu
tomek

Commits

rCOMM33c67f896a0d: [keyserver] Drop the default_role column from threads

Summary

We have now been able to remove every instance of threads.default_role from SQL queries in the database, so it should now be safe to drop this column.

The stack up until this point should be fine to land as an 'intermediate' step before introducing the admin special role

Depends on D9981

Addresses ENG-5834

Test Plan

The migration succeeded and the column was dropped. I also simulated some normal behavior on Comm and made sure no parts of the app would crash

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

rohan created this revision.Nov 24 2023, 10:34 AM

Herald added a subscriber: ashoat. · View Herald TranscriptNov 24 2023, 10:34 AM

Remove semicolon

rohan edited the summary of this revision. (Show Details)Nov 24 2023, 10:38 AM

Harbormaster completed remote builds in B24409: Diff 33608.Nov 24 2023, 11:18 AM

Harbormaster completed remote builds in B24410: Diff 33609.Nov 24 2023, 11:24 AM

rohan requested review of this revision.Nov 24 2023, 11:24 AM

tomek accepted this revision.Nov 27 2023, 3:19 AM

This revision is now accepted and ready to land.Nov 27 2023, 3:19 AM

Closed by commit rCOMM33c67f896a0d: [keyserver] Drop the default_role column from threads (authored by rohan). · Explain WhyNov 27 2023, 10:36 AM

This revision was automatically updated to reflect the committed changes.

rohan added a commit: rCOMM33c67f896a0d: [keyserver] Drop the default_role column from threads.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

database/

migration-config.js

11 lines

setup-db.js

1 line

Diff 33760

View Options

keyserver/src/database/migration-config.js