add default first in up migration

aslilac · aslilac · commit 48267a47cdad · 2024-02-12T23:19:44.000Z
diff --git a/coderd/database/migrations/000190_external_auth_providers_jsonb.down.sql b/coderd/database/migrations/000190_external_auth_providers_jsonb.down.sql
@@ -22,10 +22,10 @@ $$;
 
 
 -- Remove the non-null constraint and default
-ALTER TABLE template_versions
-  ALTER COLUMN external_auth_providers DROP NOT NULL;
 ALTER TABLE template_versions
   ALTER COLUMN external_auth_providers DROP DEFAULT;
+ALTER TABLE template_versions
+  ALTER COLUMN external_auth_providers DROP NOT NULL;
 
 
 -- Update the column type and migrate the values
diff --git a/coderd/database/migrations/000190_external_auth_providers_jsonb.up.sql b/coderd/database/migrations/000190_external_auth_providers_jsonb.up.sql
@@ -27,10 +27,10 @@ ALTER TABLE template_versions
 
 
 -- Make the column non-nullable to make the types nicer on the Go side
-ALTER TABLE template_versions
-  ALTER COLUMN external_auth_providers SET NOT NULL;
 ALTER TABLE template_versions
   ALTER COLUMN external_auth_providers SET DEFAULT '[]'::jsonb;
+ALTER TABLE template_versions
+  ALTER COLUMN external_auth_providers SET NOT NULL;
 
 
 -- Recreate `template_version_with_user` as described in dump.sql
