Finish migration to role instead of `is_admin` (#9414)

Conrad Irwin created 2 years ago

Release Notes:

- Fixed a bug signing in for five users
([#4323](https://github.com/zed-industries/zed/issues/4323)).

Change summary

crates/collab/migrations.sqlite/20221109000000_test_schema.sql           | 5 
crates/collab/migrations/20240315182903_non_null_channel_role.sql        | 4 
crates/collab/migrations/20240315183903_channel_parent_path_not_null.sql | 2 
3 files changed, 8 insertions(+), 3 deletions(-)

Detailed changes

crates/collab/migrations.sqlite/20221109000000_test_schema.sql 🔗

@@ -198,7 +198,7 @@ CREATE TABLE "channels" (
     "name" VARCHAR NOT NULL,
     "created_at" TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP,
     "visibility" VARCHAR NOT NULL,
-    "parent_path" TEXT,
+    "parent_path" TEXT NOT NULL,
     "requires_zed_cla" BOOLEAN NOT NULL DEFAULT FALSE
 );
 
@@ -237,8 +237,7 @@ CREATE TABLE "channel_members" (
     "id" INTEGER PRIMARY KEY AUTOINCREMENT,
     "channel_id" INTEGER NOT NULL REFERENCES channels (id) ON DELETE CASCADE,
     "user_id" INTEGER NOT NULL REFERENCES users (id) ON DELETE CASCADE,
-    "admin" BOOLEAN NOT NULL DEFAULT false,
-    "role" VARCHAR,
+    "role" VARCHAR NOT NULL,
     "accepted" BOOLEAN NOT NULL DEFAULT false,
     "updated_at" TIMESTAMP NOT NULL DEFAULT now
 );

crates/collab/migrations/20240315182903_non_null_channel_role.sql 🔗

@@ -0,0 +1,4 @@
+-- Add migration script here
+
+ALTER TABLE channel_members ALTER role SET NOT NULL;
+ALTER TABLE channel_members DROP COLUMN admin;

crates/collab/migrations/20240315183903_channel_parent_path_not_null.sql 🔗

@@ -0,0 +1,2 @@
+-- Add migration script here
+ALTER TABLE channels ALTER parent_path SET NOT NULL;