HomePhabricator
Diffusion Comm 33c67f896a0d

[keyserver] Drop the default_role column from threads

Description

[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

Reviewers: atul, ginsu, tomek

Reviewed By: tomek

Subscribers: ashoat

Differential Revision: https://phab.comm.dev/D9982

Details

Provenance
rohanAuthored on Nov 24 2023, 10:33 AM
Reviewer
tomek
Differential Revision
D9982: [keyserver] Drop the default_role column from threads
Parents
rCOMM6f6bf02f683d: [keyserver] Update usage of threads.default_role in thread-creator.js
Branches
Unknown
Tags
Unknown