A segment can be active or inactive #23

- Segment can be made active / inactive - migrations are added to support this - covered through test
amitavroy · Jan 4, 2025 · b825ba0 · b825ba0
1 parent f240291
commit b825ba0
Show file tree

Hide file tree

Showing 12 changed files with 159 additions and 78 deletions.
diff --git a/app/Domain/Segment/Models/Segment.php b/app/Domain/Segment/Models/Segment.php
@@ -3,6 +3,7 @@
 namespace App\Domain\Segment\Models;
 
 use Database\Factories\SegmentFactory;
+use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\SoftDeletes;
@@ -21,12 +22,19 @@ protected static function newFactory(): SegmentFactory
         'name',
         'description',
         'rules',
+        'is_active',
     ];
 
     protected function casts(): array
     {
         return [
             'rules' => 'json',
+            'is_active' => 'boolean',
         ];
     }
+
+    public function scopeActive(Builder $query): void
+    {
+        $this->where('is_active', true);
+    }
 }
diff --git a/app/Http/Controllers/HomeController.php b/app/Http/Controllers/HomeController.php
@@ -12,12 +12,11 @@
 class HomeController extends Controller
 {
     public function __invoke(
-        HomePageData     $homePageData,
-        CustomerService  $customerService,
-        ProductService   $productService,
+        HomePageData $homePageData,
+        CustomerService $customerService,
+        ProductService $productService,
         ChartDataService $chartDataService,
-    ): Response|ResponseFactory
-    {
+    ): Response|ResponseFactory {
         $data = $homePageData->handle();
 
         return inertia('Home/Index', [

diff --git a/app/Http/Controllers/SegmentController.php b/app/Http/Controllers/SegmentController.php
@@ -48,10 +48,9 @@ public function store(SegmentRequest $request): RedirectResponse
     }
 
     public function show(
-        Segment            $segment,
+        Segment $segment,
         SegmentRuleService $segmentRuleService
-    ): Response|ResponseFactory
-    {
+    ): Response|ResponseFactory {
         $rules = $segmentRuleService->getRuleNames();
 
         return inertia('Segment/Show', [
@@ -66,6 +65,7 @@ public function update(Request $request, Segment $segment): RedirectResponse
             'name' => 'required|string',
             'description' => 'required|string',
             'rules' => 'sometimes|array',
+            'is_active' => 'required|boolean',
         ]);
 
         Segment::where('id', $segment->id)
@@ -74,7 +74,5 @@ public function update(Request $request, Segment $segment): RedirectResponse
         return redirect()->route('segment.show', ['segment' => $segment->id]);
     }
 
-    public function destroy(Segment $segment): void
-    {
-    }
+    public function destroy(Segment $segment): void {}
 }
diff --git a/database/factories/SegmentFactory.php b/database/factories/SegmentFactory.php
@@ -16,8 +16,16 @@ public function definition(): array
             'name' => $this->faker->name(),
             'description' => $this->faker->text(),
             'rules' => $this->faker->words(),
+            'is_active' => true,
             'created_at' => Carbon::now(),
             'updated_at' => Carbon::now(),
         ];
     }
+
+    public function inactive(): static
+    {
+        return $this->state(fn (array $attributes) => [
+            'is_active' => false,
+        ]);
+    }
 }
diff --git a/database/migrations/2024_12_31_102036_create_segments_table.php b/database/migrations/2024_12_31_102036_create_segments_table.php
@@ -14,6 +14,7 @@ public function up(): void
             $table->string('name');
             $table->text('description');
             $table->json('rules');
+            $table->boolean('is_active')->default(true);
             $table->timestamps();
             $table->softDeletes();
         });

diff --git a/resources/js/Components/Charts/NewCustomers.vue b/resources/js/Components/Charts/NewCustomers.vue
@@ -2,13 +2,13 @@
 const { recentCustomerCount } = defineProps({
   recentCustomerCount: {
     type: Object,
-    required: true
-  }
+    required: true,
+  },
 })
 
 const total = Object.values(recentCustomerCount).reduce(
   (acc, curr) => acc + curr,
-  0
+  0,
 )
 
 const signupsChartColors = {
@@ -19,8 +19,8 @@ const signupsChartColors = {
     '#E5E7EB',
     '#E5E7EB',
     '#E5E7EB',
-    '#E5E7EB'
-  ]
+    '#E5E7EB',
+  ],
 }
 
 const options = {
@@ -31,77 +31,77 @@ const options = {
     foreColor: '#4B5563',
     fontFamily: 'Inter, sans-serif',
     toolbar: {
-      show: false
-    }
+      show: false,
+    },
   },
   theme: {
     monochrome: {
       enabled: true,
-      color: '#1A56DB'
-    }
+      color: '#1A56DB',
+    },
   },
   plotOptions: {
     bar: {
       columnWidth: '25%',
       borderRadius: 3,
       colors: {
         backgroundBarColors: signupsChartColors.backgroundBarColors,
-        backgroundBarRadius: 3
-      }
+        backgroundBarRadius: 3,
+      },
     },
     dataLabels: {
-      hideOverflowingLabels: false
-    }
+      hideOverflowingLabels: false,
+    },
   },
   xaxis: {
     floating: false,
     labels: {
-      show: false
+      show: false,
     },
     axisBorder: {
-      show: false
+      show: false,
     },
     axisTicks: {
-      show: false
-    }
+      show: false,
+    },
   },
   tooltip: {
     shared: true,
     intersect: false,
     style: {
       fontSize: '14px',
-      fontFamily: 'Inter, sans-serif'
-    }
+      fontFamily: 'Inter, sans-serif',
+    },
   },
   states: {
     hover: {
       filter: {
         type: 'darken',
-        value: 0.8
-      }
-    }
+        value: 0.8,
+      },
+    },
   },
   fill: {
-    opacity: 1
+    opacity: 1,
   },
   yaxis: {
-    show: false
+    show: false,
   },
   grid: {
-    show: false
+    show: false,
   },
   dataLabels: {
-    enabled: false
+    enabled: false,
   },
   legend: {
-    show: false
-  }
+    show: false,
+  },
 }
 const series = [
   {
     name: 'Users',
-    data: Object.values(recentCustomerCount)
-  }
+    data: Object.values(recentCustomerCount),
+  },
 ]
 </script>
 
@@ -113,7 +113,7 @@ const series = [
       <h3 class="text-base font-normal text-gray-500">New customers</h3>
       <span
         class="text-2xl font-bold leading-none text-gray-900 sm:text-3xl dark:text-white"
-      >{{ total }}</span
+        >{{ total }}</span
       >
       <p
         class="flex items-center text-base font-normal text-gray-500 dark:text-gray-400"

diff --git a/resources/js/Components/Charts/NewOrders.vue b/resources/js/Components/Charts/NewOrders.vue
@@ -2,12 +2,12 @@
 const { recentOrderData } = defineProps({
   recentOrderData: {
     type: Object,
-    required: true
+    required: true,
   },
   recentOrderCount: {
     type: Number,
-    required: true
-  }
+    required: true,
+  },
 })
 
 const options = {
@@ -18,70 +18,70 @@ const options = {
     fontFamily: 'Inter, sans-serif',
     foreColor: '#4B5563',
     toolbar: {
-      show: false
-    }
+      show: false,
+    },
   },
   plotOptions: {
     bar: {
       columnWidth: '90%',
-      borderRadius: 3
-    }
+      borderRadius: 3,
+    },
   },
   tooltip: {
     shared: false,
     intersect: false,
     style: {
       fontSize: '14px',
-      fontFamily: 'Inter, sans-serif'
-    }
+      fontFamily: 'Inter, sans-serif',
+    },
   },
   states: {
     hover: {
       filter: {
         type: 'darken',
-        value: 1
-      }
-    }
+        value: 1,
+      },
+    },
   },
   stroke: {
     show: true,
     width: 5,
-    colors: ['transparent']
+    colors: ['transparent'],
   },
   grid: {
-    show: false
+    show: false,
   },
   dataLabels: {
-    enabled: false
+    enabled: false,
   },
   legend: {
-    show: false
+    show: false,
   },
   xaxis: {
     floating: false,
     labels: {
-      show: false
+      show: false,
     },
     axisBorder: {
-      show: false
+      show: false,
     },
     axisTicks: {
-      show: false
-    }
+      show: false,
+    },
   },
   yaxis: {
-    show: false
+    show: false,
   },
   fill: {
-    opacity: 1
-  }
+    opacity: 1,
+  },
 }
 const series = [
   {
     name: 'Quantity',
     color: '#1A56DB',
-    data: recentOrderData
-  }
+    data: recentOrderData,
+  },
 ]
 </script>
 
@@ -93,7 +93,7 @@ const series = [
       <h3 class="text-base font-normal text-gray-500">New orders</h3>
       <span
         class="text-2xl font-bold leading-none text-gray-900 sm:text-3xl dark:text-white"
-      >{{ recentOrderCount }}</span
+        >{{ recentOrderCount }}</span
       >
       <p
         class="flex items-center text-base font-normal text-gray-500 dark:text-gray-400"

diff --git a/resources/js/Pages/Home/Index.vue b/resources/js/Pages/Home/Index.vue
@@ -3,7 +3,7 @@ import PageContainer from '../../Components/PageContainer.vue'
 import Breadcrumb from '../../Components/Breadcrumb.vue'
 import PageTitle from '../../Components/PageTitle.vue'
 import StatsCard from '../../Components/StatsCard.vue'
-import { usePage, usePoll } from '@inertiajs/vue3'
+import { usePoll } from '@inertiajs/vue3'
 import TopProdAndCustomer from './TopProdAndCustomer.vue'
 import NewOrders from '../../Components/Charts/NewOrders.vue'
 import NewCustomers from '../../Components/Charts/NewCustomers.vue'