From 72f5c44fad7afd4b7582500441e3b12a0457001c Mon Sep 17 00:00:00 2001
From: Daksh Shahani <dakshitshahani@gmail.com>
Date: Sat, 27 Jun 2026 20:52:05 -0700
Subject: [PATCH 1/5] edit filter function added

---
 .../features/query/popovers/filter-rows.tsx    | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/src/components/features/query/popovers/filter-rows.tsx b/src/components/features/query/popovers/filter-rows.tsx
index 7c92ec0..f8b690b 100644
--- a/src/components/features/query/popovers/filter-rows.tsx
+++ b/src/components/features/query/popovers/filter-rows.tsx
@@ -9,7 +9,7 @@ import {
   SelectValue,
 } from "@/components/ui/select";
 import { Input } from "@/components/ui/input";
-import { X, Plus } from "lucide-react";
+import { X, Plus, Pencil } from "lucide-react";
 import { Badge } from "@/components/ui/badge";
 import { Tooltip, TooltipContent, TooltipTrigger } from "@/components/ui/tooltip";
 import type { FilterRowsSelection } from "@/lib/firebase/types";
@@ -87,6 +87,14 @@ export function FilterRows({
     onRemoveFilter(filterId);
   };
 
+  const handleEditFilter = (filter: FilterRowsSelection) => {
+    setNewFilterColumn(filter.filterColumn);
+    setNewFilterCondition(filter.filterCondition);
+    setNewFilterValue(filter.filterValue);
+    setNewFilterOperator(filter.logicalOperator || 'AND');
+    onRemoveFilter(filter.id);
+  };
+
   const getFilterDisplayText = (filter: FilterRowsSelection) => {
     const conditionLabel = CONDITION_OPTIONS[columnTypes[filter.filterColumn] || "string"]
       ?.find(opt => opt.value === filter.filterCondition)?.label || filter.filterCondition;
@@ -165,6 +173,14 @@ export function FilterRows({
                     <span className="flex-1 px-2 text-xs">
                       {getFilterDisplayText(filter)}
                     </span>
+                    <Button
+                      size="icon"
+                      variant="ghost"
+                      onClick={() => handleEditFilter(filter)}
+                      className="h-6 w-6"
+                    >
+                      <Pencil className="h-3 w-3" />
+                    </Button>
                     <Button
                       size="icon"
                       variant="ghost"

From 3ec85b70fb94b139f86218a86041edacd12ba72f Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Thu, 9 Jul 2026 05:14:24 +0000
Subject: [PATCH 2/5] fix(ci): use unique Firebase preview channel per workflow
 run

---
 .github/workflows/firebase-hosting-pull-request.yml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.github/workflows/firebase-hosting-pull-request.yml b/.github/workflows/firebase-hosting-pull-request.yml
index 3bbd438..778789d 100644
--- a/.github/workflows/firebase-hosting-pull-request.yml
+++ b/.github/workflows/firebase-hosting-pull-request.yml
@@ -40,6 +40,7 @@ jobs:
         with:
           repoToken: ${{ secrets.GITHUB_TOKEN }}
           firebaseServiceAccount: ${{ secrets.FIREBASE_SERVICE_ACCOUNT_NWPLUS_UBC_DEV }}
+          channelId: pr${{ github.event.pull_request.number }}-${{ github.run_id }}
           expires: 10d
           projectId: nwplus-ubc-dev
           target: admin-portal-dev

From 234072fa48367e49fd909d868d49b7208662d28c Mon Sep 17 00:00:00 2001
From: Daksh Shahani <dakshitshahani@gmail.com>
Date: Wed, 8 Jul 2026 17:48:59 -0700
Subject: [PATCH 3/5] feat(query): add in-place onFilterUpdate mutator to
 QueryProvider

Adds handleFilterUpdate that replaces a filter selection in-place by ID using the existing .map(). Exposes it as onFilterUpdate in QueryContextType and the provider value object.
---
 src/providers/query-provider.tsx | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/providers/query-provider.tsx b/src/providers/query-provider.tsx
index 70d4da9..cb3c990 100644
--- a/src/providers/query-provider.tsx
+++ b/src/providers/query-provider.tsx
@@ -127,6 +127,7 @@ interface QueryContextType {
   onFilterAdd: (selection: FilterRowsSelection) => void;
   onFilterRemove: (filterId: string) => void;
   onFilterOperatorChange: (filterId: string, operator: 'AND' | 'OR') => void;
+  onFilterUpdate: (filterId: string, selection: FilterRowsSelection) => void;
   onSortingChange: (updater: SortingState | ((prev: SortingState) => SortingState)) => void;
 }
 
@@ -246,6 +247,12 @@ export function QueryProvider({ children }: QueryProviderProps) {
     ));
   }, []);
 
+  const handleFilterUpdate = useCallback((filterId: string, selection: FilterRowsSelection) => {
+    setFilterSelections(prev => prev.map(filter =>
+      filter.id === filterId ? { ...selection, id: filterId } : filter
+    ));
+  }, []);
+
   const value = {
     hackathons,
     selectedHackathon,
@@ -262,6 +269,7 @@ export function QueryProvider({ children }: QueryProviderProps) {
     onFilterAdd: handleFilterAdd,
     onFilterRemove: handleFilterRemove,
     onFilterOperatorChange: handleFilterOperatorChange,
+    onFilterUpdate: handleFilterUpdate,
     onSortingChange: setSorting,
   };
 

From 678285dfcb58555c7f91226342c18a291ff71342 Mon Sep 17 00:00:00 2001
From: Daksh Shahani <dakshitshahani@gmail.com>
Date: Wed, 8 Jul 2026 22:10:44 -0700
Subject: [PATCH 4/5] fix(query): edit filter in place instead of
 remove-and-re-add

---
 .../features/query/popovers/filter-rows.tsx   | 69 +++++++++++++++----
 1 file changed, 54 insertions(+), 15 deletions(-)

diff --git a/src/components/features/query/popovers/filter-rows.tsx b/src/components/features/query/popovers/filter-rows.tsx
index f8b690b..eca9f8f 100644
--- a/src/components/features/query/popovers/filter-rows.tsx
+++ b/src/components/features/query/popovers/filter-rows.tsx
@@ -9,7 +9,7 @@ import {
   SelectValue,
 } from "@/components/ui/select";
 import { Input } from "@/components/ui/input";
-import { X, Plus, Pencil } from "lucide-react";
+import { X, Plus, Pencil, Check } from "lucide-react";
 import { Badge } from "@/components/ui/badge";
 import { Tooltip, TooltipContent, TooltipTrigger } from "@/components/ui/tooltip";
 import type { FilterRowsSelection } from "@/lib/firebase/types";
@@ -42,6 +42,7 @@ interface FilterRowsProps {
   filterSelections: FilterRowsSelection[];
   onAddFilter: (filter: FilterRowsSelection) => void;
   onRemoveFilter: (filterId: string) => void;
+  onFilterUpdate?: (filterId: string, filter: FilterRowsSelection) => void;
   onFilterOperatorChange: (filterId: string, operator: 'AND' | 'OR') => void;
 }
 
@@ -55,6 +56,7 @@ export function FilterRows({
   filterSelections, 
   onAddFilter, 
   onRemoveFilter,
+  onFilterUpdate,
   onFilterOperatorChange
 }: FilterRowsProps) {
   const [open, setOpen] = useState(false);
@@ -62,24 +64,37 @@ export function FilterRows({
   const [newFilterCondition, setNewFilterCondition] = useState("");
   const [newFilterValue, setNewFilterValue] = useState("");
   const [newFilterOperator, setNewFilterOperator] = useState<'AND' | 'OR'>('AND');
+  const [editingFilterId, setEditingFilterId] = useState<string | null>(null);
 
   const type = columnTypes[newFilterColumn] || "string";
   const conditionOptions = CONDITION_OPTIONS[type] || CONDITION_OPTIONS.string;
 
-  const handleAddFilter = () => {
+  const handleCommitFilter = () => {
     if (newFilterColumn && newFilterCondition && newFilterValue) {
-      const newFilter: FilterRowsSelection = {
-        id: crypto.randomUUID(),
-        filterColumn: newFilterColumn,
-        filterCondition: newFilterCondition,
-        filterValue: newFilterValue,
-        logicalOperator: newFilterOperator,
-      };
-      onAddFilter(newFilter);
+      if (editingFilterId) {
+        const updatedFilter: FilterRowsSelection = {
+          id: editingFilterId,
+          filterColumn: newFilterColumn,
+          filterCondition: newFilterCondition,
+          filterValue: newFilterValue,
+          logicalOperator: newFilterOperator,
+        };
+        onFilterUpdate?.(editingFilterId, updatedFilter);
+      } else {
+        const newFilter: FilterRowsSelection = {
+          id: crypto.randomUUID(),
+          filterColumn: newFilterColumn,
+          filterCondition: newFilterCondition,
+          filterValue: newFilterValue,
+          logicalOperator: newFilterOperator,
+        };
+        onAddFilter(newFilter);
+      }
       setNewFilterColumn("");
       setNewFilterCondition("");
       setNewFilterValue("");
       setNewFilterOperator('AND');
+      setEditingFilterId(null);
     }
   };
 
@@ -92,7 +107,7 @@ export function FilterRows({
     setNewFilterCondition(filter.filterCondition);
     setNewFilterValue(filter.filterValue);
     setNewFilterOperator(filter.logicalOperator || 'AND');
-    onRemoveFilter(filter.id);
+    setEditingFilterId(filter.id);
   };
 
   const getFilterDisplayText = (filter: FilterRowsSelection) => {
@@ -114,7 +129,16 @@ export function FilterRows({
   };
 
   return (
-    <Popover open={open} onOpenChange={setOpen}>
+    <Popover open={open} onOpenChange={(isOpen) => {
+      if (!isOpen) {
+        setNewFilterColumn("");
+        setNewFilterCondition("");
+        setNewFilterValue("");
+        setNewFilterOperator('AND');
+        setEditingFilterId(null);
+      }
+      setOpen(isOpen);
+    }}>
       <PopoverTrigger asChild>
         <div className="flex items-center gap-2">
           <Button variant="outline" className="w-full justify-between font-normal">
@@ -197,7 +221,7 @@ export function FilterRows({
         )}
 
         <div className="mt-4 space-y-2">
-          <h4 className="font-medium text-sm">Add New Filter</h4>
+          <h4 className="font-medium text-sm">{editingFilterId ? "Edit Filter" : "Add New Filter"}</h4>
           <div className="space-y-2">
             {filterSelections.length > 0 && (
               <div className="flex items-center gap-2">
@@ -259,11 +283,26 @@ export function FilterRows({
               <Button
                 size="icon"
                 variant="ghost"
-                onClick={handleAddFilter}
+                onClick={handleCommitFilter}
                 disabled={!(newFilterColumn && newFilterCondition && newFilterValue)}
               >
-                <Plus className="h-4 w-4" />
+                {editingFilterId ? <Check className="h-4 w-4" /> : <Plus className="h-4 w-4" />}
               </Button>
+              {editingFilterId && (
+                <Button
+                  size="icon"
+                  variant="ghost"
+                  onClick={() => {
+                    setNewFilterColumn("");
+                    setNewFilterCondition("");
+                    setNewFilterValue("");
+                    setNewFilterOperator('AND');
+                    setEditingFilterId(null);
+                  }}
+                >
+                  <X className="h-4 w-4" />
+                </Button>
+              )}
             </div>
           </div>
         </div>

From dc2e35ce149931fe7628c0689dd1af635e8874ea Mon Sep 17 00:00:00 2001
From: Daksh Shahani <dakshitshahani@gmail.com>
Date: Wed, 8 Jul 2026 22:15:29 -0700
Subject: [PATCH 5/5] chore(query): wire onFilterUpdate through query-filters
 consumer

---
 src/components/features/query/popovers/filter-rows.tsx | 4 ++--
 src/components/features/query/query-filters.tsx        | 2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/components/features/query/popovers/filter-rows.tsx b/src/components/features/query/popovers/filter-rows.tsx
index eca9f8f..e87e89e 100644
--- a/src/components/features/query/popovers/filter-rows.tsx
+++ b/src/components/features/query/popovers/filter-rows.tsx
@@ -42,7 +42,7 @@ interface FilterRowsProps {
   filterSelections: FilterRowsSelection[];
   onAddFilter: (filter: FilterRowsSelection) => void;
   onRemoveFilter: (filterId: string) => void;
-  onFilterUpdate?: (filterId: string, filter: FilterRowsSelection) => void;
+  onFilterUpdate: (filterId: string, filter: FilterRowsSelection) => void;
   onFilterOperatorChange: (filterId: string, operator: 'AND' | 'OR') => void;
 }
 
@@ -79,7 +79,7 @@ export function FilterRows({
           filterValue: newFilterValue,
           logicalOperator: newFilterOperator,
         };
-        onFilterUpdate?.(editingFilterId, updatedFilter);
+        onFilterUpdate(editingFilterId, updatedFilter);
       } else {
         const newFilter: FilterRowsSelection = {
           id: crypto.randomUUID(),
diff --git a/src/components/features/query/query-filters.tsx b/src/components/features/query/query-filters.tsx
index 190e6aa..e52dd7d 100644
--- a/src/components/features/query/query-filters.tsx
+++ b/src/components/features/query/query-filters.tsx
@@ -18,6 +18,7 @@ export function QueryFilters({ availableColumns }: QueryFiltersProps) {
     filterSelections,
     onFilterAdd,
     onFilterRemove,
+    onFilterUpdate,
     onFilterOperatorChange,
     sorting,
     onSortingChange,
@@ -115,6 +116,7 @@ export function QueryFilters({ availableColumns }: QueryFiltersProps) {
             filterSelections={filterSelections}
             onAddFilter={onFilterAdd}
             onRemoveFilter={onFilterRemove}
+            onFilterUpdate={onFilterUpdate}
             onFilterOperatorChange={onFilterOperatorChange}
           />
         </div>