From 318ebdfd26331ca12ccf8ca77fe751a6826c8d90 Mon Sep 17 00:00:00 2001
From: Nabarun Gogoi <nabarun@deepstacksoft.com>
Date: Wed, 20 Dec 2023 16:43:27 +0530
Subject: [PATCH] Filter deployments with status options (#15)

---
 .../frontend/src/components/SearchBar.tsx     | 14 +---
 .../projects/project/DeploymentsTabPanel.tsx  | 75 ++++++++----------
 .../DeploymentDetailsCard.tsx                 |  2 +-
 .../project/deployments/FilterForm.tsx        | 77 +++++++++++++++++++
 4 files changed, 114 insertions(+), 54 deletions(-)
 rename packages/frontend/src/components/projects/project/{ => deployments}/DeploymentDetailsCard.tsx (96%)
 create mode 100644 packages/frontend/src/components/projects/project/deployments/FilterForm.tsx

diff --git a/packages/frontend/src/components/SearchBar.tsx b/packages/frontend/src/components/SearchBar.tsx
index 473da86..95e6ec9 100644
--- a/packages/frontend/src/components/SearchBar.tsx
+++ b/packages/frontend/src/components/SearchBar.tsx
@@ -1,16 +1,10 @@
-import React, { ChangeEventHandler, forwardRef } from 'react';
+import React, { forwardRef } from 'react';
 
-import { Input } from '@material-tailwind/react';
-
-interface SearchBarProps {
-  onChange?: ChangeEventHandler<HTMLInputElement>;
-  value?: string;
-  placeholder?: string;
-}
+import { Input, InputProps } from '@material-tailwind/react';
 
 const SearchBar: React.ForwardRefRenderFunction<
   HTMLInputElement,
-  SearchBarProps
+  InputProps
 > = ({ value, onChange, placeholder = 'Search', ...props }, ref) => {
   return (
     <div className="relative flex w-full gap-2">
@@ -28,8 +22,8 @@ const SearchBar: React.ForwardRefRenderFunction<
         }}
         // TODO: Debug issue: https://github.com/creativetimofficial/material-tailwind/issues/427
         crossOrigin={undefined}
-        ref={ref}
         {...props}
+        ref={ref}
       />
       <div className="!absolute left-3 top-[13px]">
         <i>^</i>
diff --git a/packages/frontend/src/components/projects/project/DeploymentsTabPanel.tsx b/packages/frontend/src/components/projects/project/DeploymentsTabPanel.tsx
index 906e70d..214a92c 100644
--- a/packages/frontend/src/components/projects/project/DeploymentsTabPanel.tsx
+++ b/packages/frontend/src/components/projects/project/DeploymentsTabPanel.tsx
@@ -1,60 +1,47 @@
 import React, { useCallback, useMemo, useState } from 'react';
 
-import { Button } from '@material-tailwind/react';
+import { Button, Typography } from '@material-tailwind/react';
 
 import deploymentData from '../../../assets/deployments.json';
-import DeployDetailsCard from './DeploymentDetailsCard';
-import Dropdown from '../../Dropdown';
-import SearchBar from '../../SearchBar';
+import DeployDetailsCard from './deployments/DeploymentDetailsCard';
+import FilterForm, { StatusOptions } from './deployments/FilterForm';
 
-const STATUS_OPTIONS = [
-  { value: 'building', label: 'Building' },
-  { value: 'ready', label: 'Ready' },
-  { value: 'error', label: 'Error' },
-];
+const DEFAULT_FILTER_VALUE = {
+  searchedBranch: '',
+  status: 'All status',
+};
 
 const DeploymentsTabPanel = () => {
-  const [searchedBranch, setSearchedBranch] = useState('');
+  const [filterValue, setFilterValue] = useState(DEFAULT_FILTER_VALUE);
 
   const filteredDeployments = useMemo(() => {
-    if (searchedBranch) {
-      return deploymentData.filter((deployment) =>
-        deployment.branch.toLowerCase().includes(searchedBranch.toLowerCase()),
-      );
-    }
+    return deploymentData.filter((deployment) => {
+      // Searched branch filter
+      const branchMatch =
+        !filterValue.searchedBranch ||
+        deployment.branch
+          .toLowerCase()
+          .includes(filterValue.searchedBranch.toLowerCase());
 
-    return deploymentData;
-  }, [searchedBranch]);
+      // Status filter
+      const statusMatch =
+        filterValue.status === StatusOptions.ALL_STATUS ||
+        deployment.status === filterValue.status;
+
+      return branchMatch && statusMatch;
+    });
+  }, [filterValue]);
 
   const handleResetFilters = useCallback(() => {
-    setSearchedBranch('');
+    setFilterValue(DEFAULT_FILTER_VALUE);
   }, []);
 
   return (
     <div className="p-4">
-      <div className="grid grid-cols-4 gap-2 text-sm text-gray-600">
-        <div className="col-span-2">
-          <SearchBar
-            placeholder="Search branches"
-            value={searchedBranch}
-            onChange={(event) => setSearchedBranch(event.target.value)}
-          />
-        </div>
-        <div className="col-span-1">
-          <input
-            type="text"
-            className="border border-gray-300 rounded p-2 w-full focus:border-blue-300 focus:outline-none focus:shadow-outline-blue"
-            placeholder="All time"
-          />
-        </div>
-        <div className="col-span-1">
-          <Dropdown
-            placeholder="All status"
-            options={STATUS_OPTIONS}
-            onChange={() => {}}
-          />
-        </div>
-      </div>
+      <FilterForm
+        value={filterValue}
+        onChange={(value) => setFilterValue(value)}
+      />
       <div className="mt-2">
         {Boolean(filteredDeployments.length) ? (
           filteredDeployments.map((deployment, key) => {
@@ -63,8 +50,10 @@ const DeploymentsTabPanel = () => {
         ) : (
           <div className="h-[50vh] bg-gray-100 flex rounded items-center justify-center">
             <div className="text-center">
-              <h5 className="text-lg font-bold">No deployments found</h5>
-              <p>Please change your search query or filters</p>
+              <Typography variant="h5">No deployments found</Typography>
+              <Typography>
+                Please change your search query or filters
+              </Typography>
               <Button
                 className="rounded-full mt-5"
                 color="white"
diff --git a/packages/frontend/src/components/projects/project/DeploymentDetailsCard.tsx b/packages/frontend/src/components/projects/project/deployments/DeploymentDetailsCard.tsx
similarity index 96%
rename from packages/frontend/src/components/projects/project/DeploymentDetailsCard.tsx
rename to packages/frontend/src/components/projects/project/deployments/DeploymentDetailsCard.tsx
index 3da0405..d1d7e54 100644
--- a/packages/frontend/src/components/projects/project/DeploymentDetailsCard.tsx
+++ b/packages/frontend/src/components/projects/project/deployments/DeploymentDetailsCard.tsx
@@ -7,7 +7,7 @@ import {
   MenuItem,
 } from '@material-tailwind/react';
 
-import { relativeTime } from '../../../utils/time';
+import { relativeTime } from '../../../../utils/time';
 
 interface DeploymentDetails {
   title: string;
diff --git a/packages/frontend/src/components/projects/project/deployments/FilterForm.tsx b/packages/frontend/src/components/projects/project/deployments/FilterForm.tsx
new file mode 100644
index 0000000..26366ab
--- /dev/null
+++ b/packages/frontend/src/components/projects/project/deployments/FilterForm.tsx
@@ -0,0 +1,77 @@
+import React, { useEffect, useState } from 'react';
+
+import { Option, Select } from '@material-tailwind/react';
+
+import SearchBar from '../../../SearchBar';
+
+export enum StatusOptions {
+  ALL_STATUS = 'All status',
+  BUILDING = 'Building',
+  READY = 'Ready',
+  ERROR = 'Error',
+}
+
+interface FilterValue {
+  searchedBranch: string;
+  status: string;
+}
+
+interface FilterFormProps {
+  value: FilterValue;
+  onChange: (value: FilterValue) => void;
+}
+
+const FilterForm = ({ value, onChange }: FilterFormProps) => {
+  const [searchedBranch, setSearchedBranch] = useState(value.searchedBranch);
+  const [selectedStatus, setSelectedStatus] = useState(value.status);
+
+  useEffect(() => {
+    onChange({
+      searchedBranch,
+      status: selectedStatus,
+    });
+  }, [searchedBranch, selectedStatus]);
+
+  useEffect(() => {
+    setSearchedBranch(value.searchedBranch);
+    setSelectedStatus(value.status);
+  }, [value]);
+
+  return (
+    <div className="grid grid-cols-4 gap-2 text-sm text-gray-600">
+      <div className="col-span-2">
+        <SearchBar
+          placeholder="Search branches"
+          value={searchedBranch}
+          onChange={(event) => setSearchedBranch(event.target.value)}
+        />
+      </div>
+      <div className="col-span-1">
+        <input
+          type="text"
+          className="border border-gray-300 rounded p-2 w-full focus:border-blue-300 focus:outline-none focus:shadow-outline-blue"
+          placeholder="All time"
+        />
+      </div>
+      <div className="col-span-1">
+        <Select
+          value={selectedStatus}
+          onChange={(value) => setSelectedStatus(value!)}
+          label="Select Version"
+        >
+          {Object.values(StatusOptions).map((status) => (
+            <Option
+              className={status === selectedStatus ? 'hidden' : ''}
+              key={status}
+              value={status}
+            >
+              ^ {status}
+            </Option>
+          ))}
+        </Select>
+      </div>
+    </div>
+  );
+};
+
+export default FilterForm;