Merge pull request #2266 from bluewave-labs/hp-oct-1-fix-project-status-migration-bug

Fix project status migration bug

Author: MuhammadKhalilzadeh

Servers/database/migrations/20250922000001-add-project-status.js

@@ -11,36 +11,26 @@ module.exports = {
         `SELECT id FROM public.organizations;`, { transaction }
       );
 
-      // Add status column to public.projects table
-      await queryInterface.addColumn(
-        'projects',
-        'status',
-        {
-          type: Sequelize.ENUM('Not started', 'In progress', 'Under review', 'Completed', 'Closed', 'On hold', 'Rejected'),
-          allowNull: false,
-          defaultValue: 'Not started'
-        },
-        { transaction }
-      );
+      await queryInterface.sequelize.query(
+        `CREATE TYPE projects_status_enum AS ENUM (
+          'Not started',
+          'In progress',
+          'Under review',
+          'Completed',
+          'Closed',
+          'On hold',
+          'Rejected'
+        );`, { transaction });
 
       // Add status column to each tenant's projects table
       for (let organization of organizations[0]) {
         const tenantHash = getTenantHash(organization.id);
-        
+
         // Add the column as nullable first
-        await queryInterface.addColumn(
-          {
-            tableName: 'projects',
-            schema: tenantHash
-          },
-          'status',
-          {
-            type: Sequelize.ENUM('Not started', 'In progress', 'Under review', 'Completed', 'Closed', 'On hold', 'Rejected'),
-            allowNull: true,
-            defaultValue: 'Not started'
-          },
-          { transaction }
-        );
+        await queryInterface.sequelize.query(`
+          ALTER TABLE "${tenantHash}".projects
+            ADD COLUMN "status" projects_status_enum
+            DEFAULT 'Not started';`, { transaction });
 
         // Update all existing records to have the default status
         await queryInterface.sequelize.query(
@@ -49,19 +39,9 @@ module.exports = {
         );
 
         // Then change the column to not allow null
-        await queryInterface.changeColumn(
-          {
-            tableName: 'projects',
-            schema: tenantHash
-          },
-          'status',
-          {
-            type: Sequelize.ENUM('Not started', 'In progress', 'Under review', 'Completed', 'Closed', 'On hold', 'Rejected'),
-            allowNull: false,
-            defaultValue: 'Not started'
-          },
-          { transaction }
-        );
+        await queryInterface.sequelize.query(`
+          ALTER TABLE "${tenantHash}".projects
+            ALTER COLUMN "status" SET NOT NULL;`, { transaction });
       }
 
       await transaction.commit();
@@ -74,9 +54,6 @@ module.exports = {
   async down(queryInterface, Sequelize) {
     const transaction = await queryInterface.sequelize.transaction();
     try {
-      // Remove status column from public.projects table
-      await queryInterface.removeColumn('projects', 'status', { transaction });
-
       // Get all organizations to update their tenant schemas
       const organizations = await queryInterface.sequelize.query(
         `SELECT id FROM public.organizations;`, { transaction }
@@ -85,7 +62,7 @@ module.exports = {
       // Remove status column from each tenant's projects table
       for (let organization of organizations[0]) {
         const tenantHash = getTenantHash(organization.id);
-        
+
         await queryInterface.removeColumn(
           {
             tableName: 'projects',

Servers/scripts/createNewTenant.ts

@@ -41,7 +41,7 @@ export const createNewTenant = async (organization_id: number, transaction: Tran
         last_updated_by integer,
         is_demo boolean NOT NULL DEFAULT false,
         is_organizational boolean NOT NULL DEFAULT false,
-        status enum_projects_status NOT NULL DEFAULT 'Not started',
+        status projects_status_enum NOT NULL DEFAULT 'Not started',
         created_at timestamp without time zone NOT NULL DEFAULT now(),
         CONSTRAINT projects_pkey PRIMARY KEY (id),
         CONSTRAINT projects_owner_fkey FOREIGN KEY (owner)