Adds enabled to project to hide disabled projects from new incidents and cases (Netflix#4572)

whitdog47 · web-flow · commit ad25225452d5 · 2024-04-05T14:12:20.000-07:00
diff --git a/src/dispatch/database/revisions/tenant/versions/2024-04-02_71cb25c06fa0.py b/src/dispatch/database/revisions/tenant/versions/2024-04-02_71cb25c06fa0.py
@@ -0,0 +1,27 @@
+"""Adds enabled to project
+
+Revision ID: 71cb25c06fa0
+Revises: 91bd05855ad1
+Create Date: 2024-04-02 12:31:20.012288
+
+"""
+from alembic import op
+import sqlalchemy as sa
+
+# revision identifiers, used by Alembic.
+revision = "71cb25c06fa0"
+down_revision = "91bd05855ad1"
+branch_labels = None
+depends_on = None
+
+
+def upgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.add_column("project", sa.Column("enabled", sa.Boolean(), server_default="t", nullable=True, default=True))
+    # ### end Alembic commands ###
+
+
+def downgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_column("project", "enabled")
+    # ### end Alembic commands ###
diff --git a/src/dispatch/project/models.py b/src/dispatch/project/models.py
@@ -39,6 +39,8 @@ class Project(Base):
         cascade="all, delete-orphan",
     )
 
+    enabled = Column(Boolean, default=True, server_default="t")
+
     send_daily_reports = Column(Boolean)
 
     stable_priority_id = Column(Integer, nullable=True)
@@ -68,6 +70,7 @@ class ProjectBase(DispatchBase):
     default: bool = False
     color: Optional[str] = Field(None, nullable=True)
     send_daily_reports: Optional[bool] = Field(True, nullable=True)
+    enabled: Optional[bool] = Field(True, nullable=True)
 
 
 class ProjectCreate(ProjectBase):
diff --git a/src/dispatch/static/dispatch/src/case/ReportSubmissionCard.vue b/src/dispatch/static/dispatch/src/case/ReportSubmissionCard.vue
@@ -60,7 +60,7 @@
               />
             </v-col>
             <v-col cols="12">
-              <project-select v-model="project" />
+              <project-select v-model="project" excludeDisabled />
             </v-col>
             <v-col cols="12">
               <case-type-select :project="project" v-model="case_type" />
diff --git a/src/dispatch/static/dispatch/src/incident/ReportSubmissionCard.vue b/src/dispatch/static/dispatch/src/incident/ReportSubmissionCard.vue
@@ -57,7 +57,7 @@
             />
           </v-col>
           <v-col cols="12">
-            <project-select v-model="project" />
+            <project-select v-model="project" excludeDisabled />
           </v-col>
           <v-col cols="12">
             <incident-type-select :project="project" v-model="incident_type" />
diff --git a/src/dispatch/static/dispatch/src/project/NewEditSheet.vue b/src/dispatch/static/dispatch/src/project/NewEditSheet.vue
@@ -52,6 +52,13 @@
                   name="Description"
                 />
               </v-col>
+              <v-col cols="12">
+                <v-checkbox
+                  v-model="enabled"
+                  label="Enabled"
+                  hint="Whether this project is enabled for new cases and incidents."
+                />
+              </v-col>
               <v-col cols="12">
                 <color-picker-input v-model="color" />
               </v-col>
@@ -144,6 +151,7 @@ export default {
       "selected.organization",
       "selected.owner_conversation",
       "selected.owner_email",
+      "selected.enabled",
       "dialogs.showCreateEdit",
     ]),
   },
diff --git a/src/dispatch/static/dispatch/src/project/ProjectSelect.vue b/src/dispatch/static/dispatch/src/project/ProjectSelect.vue
@@ -36,6 +36,7 @@
 <script>
 import { cloneDeep, debounce } from "lodash"
 import ProjectApi from "@/project/api"
+import SearchUtils from "@/search/utils"
 
 export default {
   name: "ProjectSelect",
@@ -47,6 +48,10 @@ export default {
         return {}
       },
     },
+    excludeDisabled: {
+      type: Boolean,
+      default: false,
+    },
     label: {
       type: String,
       default: "Project",
@@ -66,7 +71,13 @@ export default {
   computed: {
     project: {
       get() {
-        return cloneDeep(this.modelValue)
+        let projects = cloneDeep(this.modelValue)
+        if (this.excludeDisabled && projects && Array.isArray(projects) && projects.length > 0) {
+          projects = projects.filter((project) => {
+            return project.enabled
+          })
+        }
+        return projects
       },
       set(value) {
         this.$emit("update:modelValue", value)
@@ -89,6 +100,17 @@ export default {
         descending: [false],
       }
 
+      if (this.excludeDisabled) {
+        filterOptions = {
+          filters: {
+            enabled: [true],
+          },
+          ...filterOptions,
+        }
+      }
+
+      filterOptions = SearchUtils.createParametersFromTableOptions({ ...filterOptions })
+
       ProjectApi.getAll(filterOptions).then((response) => {
         this.items = response.data.items
 
diff --git a/src/dispatch/static/dispatch/src/project/Table.vue b/src/dispatch/static/dispatch/src/project/Table.vue
@@ -39,6 +39,9 @@
                 {{ item.name }}
               </router-link>
             </template>
+            <template #item.enabled="{ value }">
+              <v-checkbox-btn :model-value="value" disabled />
+            </template>
             <template #item.data-table-actions="{ item }">
               <v-menu location="right" origin="overlap">
                 <template #activator="{ props }">
@@ -85,6 +88,7 @@ export default {
         { title: "Description", value: "description", sortable: false },
         { title: "Annual Employee Cost", value: "annual_employee_cost", sortable: false },
         { title: "Business Year Hours", value: "business_year_hours", sortable: false },
+        { title: "Enabled", value: "enabled", sortable: false },
         { title: "", key: "data-table-actions", sortable: false, align: "end" },
       ],
     }
diff --git a/src/dispatch/static/dispatch/src/project/store.js b/src/dispatch/static/dispatch/src/project/store.js
@@ -15,6 +15,7 @@ const getDefaultSelectedState = () => {
     business_year_hours: 2080,
     owner_email: null,
     owner_conversation: null,
+    enabled: null,
   }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -15,6 +15,7 @@ const getDefaultSelectedState = () => {`
`15`	`15`	`business_year_hours: 2080,`
`16`	`16`	`owner_email: null,`
`17`	`17`	`owner_conversation: null,`
	`18`	`+ enabled: null,`
`18`	`19`	`}`
`19`	`20`	`}`
`20`	`21`