From 38e1819336873fa6c3aa1ccbb6f5b34581d9935d Mon Sep 17 00:00:00 2001
From: Luke C Hartman <luke@lukecarlhartman.com>
Date: Tue, 22 Oct 2024 11:22:20 -0700
Subject: [PATCH 1/7] defaultLabels fix & admin user can edit labels

---
 components/ReportSystem.jsx          |  2 +-
 components/Settings.jsx              | 14 ++++++++++++--
 components/TagSystem.jsx             |  2 +-
 components/modals/NewReportModal.jsx |  4 ++--
 4 files changed, 16 insertions(+), 6 deletions(-)

diff --git a/components/ReportSystem.jsx b/components/ReportSystem.jsx
index cb0a39a..d769bd8 100644
--- a/components/ReportSystem.jsx
+++ b/components/ReportSystem.jsx
@@ -109,7 +109,7 @@ const ReportSystem = ({
 
   const defaultTopics = ["Health","Other","Politics","Weather"] // tag system 1
   const defaultSources = ["Newspaper", "Other","Social","Website"] // tag system 2
-  const defaultLabels = ["Important", "Flagged"] // tag system 3
+  const defaultLabels = ["To Investigate", "Investigated: Flagged", "Investigated: Benign"] // tag system 3
 
 	// useEffect(() => {
 	// 	console.log('Authenticated user:', user);
diff --git a/components/Settings.jsx b/components/Settings.jsx
index d9b9e14..d5b3488 100644
--- a/components/Settings.jsx
+++ b/components/Settings.jsx
@@ -29,7 +29,7 @@ const Settings = () => {
   const setData = async(agencyID) => {
     const defaultTopics = ["Health","Other","Politics","Weather"] // tag system 1
     const defaultSources = ["Newspaper", "Other/Otro","Social","Website"] // tag system 2
-    const defaultLabels = ["Important", "Flagged"] // tag system 3
+    const defaultLabels = ["To Investigate", "Investigated: Flagged", "Investigated: Benign"] // tag system 3
 
     // create topics collection for the new agency
     setDoc(doc(db, "tags", agencyID), {
@@ -181,7 +181,7 @@ const Settings = () => {
         </div>
         }
         {agencyID && 
-        <div>
+      <div>
         <div className="flex justify-between mx-6 my-6 tracking-normal items-center">
             <div className="font-light">Topic Tags</div>
             <button
@@ -198,6 +198,16 @@ const Settings = () => {
                 Edit Sources
             </button>
         </div>
+        {customClaims.admin &&
+          <div className="flex justify-between mx-6 my-6 tracking-normal items-center">
+            <div className="font-light">Labels</div>
+            <button
+                onClick={() => setTagSystem(3)}
+                className="bg-sky-100 hover:bg-blue-200 text-blue-600 font-normal py-2 px-6 border border-blue-600 rounded-xl">
+                Edit Labels
+            </button>
+          </div>
+        }
         {/* <div className="flex justify-between mx-6 my-6 tracking-normal items-center">
             <div className="font-light">Customized Labels</div>
             <button
diff --git a/components/TagSystem.jsx b/components/TagSystem.jsx
index a077547..f25b407 100644
--- a/components/TagSystem.jsx
+++ b/components/TagSystem.jsx
@@ -53,7 +53,7 @@ const TagSystem = ({ tagSystem, setTagSystem, agencyID}) => {
 
     const defaultTopics = ["Health","Other","Politics","Weather"] // tag system 1
     const defaultSources = ["Newspaper", "Other","Social","Website"] // tag system 2
-    const defaultLabels = ["Important", "Flagged"] // tag system 3
+    const defaultLabels = ["To Investigate", "Investigated: Flagged", "Investigated: Benign"] // tag system 3
 
     const tags = ["Topic", "Source", "Labels"]
     //const docRef = getDoc(db, "tags", tagSystem)
diff --git a/components/modals/NewReportModal.jsx b/components/modals/NewReportModal.jsx
index ff49bd4..5718a15 100644
--- a/components/modals/NewReportModal.jsx
+++ b/components/modals/NewReportModal.jsx
@@ -311,7 +311,7 @@ const NewReportModal = ({
           // Create default tags if they don't exist
           const defaultTopics = ['Health', 'Other', 'Politics', 'Weather']
           const defaultSources = ['Newspaper', 'Other/Otro', 'Social', 'Website']
-          const defaultLabels = ['Important', 'Flagged']
+          const defaultLabels = ["To Investigate", "Investigated: Flagged", "Investigated: Benign"]
 
           await setDoc(docRef, {
             Labels: { list: defaultLabels, active: defaultLabels },
@@ -488,7 +488,7 @@ const NewReportModal = ({
 				console.log('Need to create tag collection for agency. ')
 				const defaultTopics = ['Health', 'Other', 'Politics', 'Weather'] // tag system 1
 				const defaultSources = ['Newspaper', 'Other/Otro', 'Social', 'Website'] // tag system 2
-				const defaultLabels = ['Important', 'Flagged'] // tag system 3
+				const defaultLabels = ["To Investigate", "Investigated: Flagged", "Investigated: Benign"] // tag system 3
 
 				// reference to tags collection
 				const myDocRef = doc(db, 'tags', selectedAgencyId)

From 9e9080c48911038071df12eec1b784d4fb3776c7 Mon Sep 17 00:00:00 2001
From: Luke C Hartman <luke@lukecarlhartman.com>
Date: Tue, 22 Oct 2024 11:29:49 -0700
Subject: [PATCH 2/7] tag length 25 chars

---
 components/modals/RenameTagModal.jsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/components/modals/RenameTagModal.jsx b/components/modals/RenameTagModal.jsx
index 50e99c1..0a8ca71 100644
--- a/components/modals/RenameTagModal.jsx
+++ b/components/modals/RenameTagModal.jsx
@@ -53,7 +53,7 @@ const RenameTagModal = ({ replaceTag, selected, list, setRenameTagModal, addNewT
                                 />
                         </div>
                         {lower.includes(tag.toLowerCase()) && <p className="text-red-500 text-sm font-light">This tag already exists, do you want to replace?</p>}
-                        {tag.length > 20 && <p className="text-red-500 text-sm font-light">You cannot type a tag more than 18 characters long. Please try another name</p>}
+                        {tag.length > 25 && <p className="text-red-500 text-sm font-light">You cannot type a tag more than 18 characters long. Please try another name</p>}
                         <div className="mt-6 flex justify-between">
                             <button
                                 onClick={handleReplaceTag}

From ee059ae35d8f6e7523e63cf7bccedc3d3bfd20da Mon Sep 17 00:00:00 2001
From: Luke C Hartman <luke@lukecarlhartman.com>
Date: Tue, 22 Oct 2024 11:56:43 -0700
Subject: [PATCH 3/7] error logging for replace tag

---
 components/TagSystem.jsx             | 13 +++++++++----
 components/modals/RenameTagModal.jsx | 28 +++++++++++++++++++++-------
 2 files changed, 30 insertions(+), 11 deletions(-)

diff --git a/components/TagSystem.jsx b/components/TagSystem.jsx
index f25b407..884b0a5 100644
--- a/components/TagSystem.jsx
+++ b/components/TagSystem.jsx
@@ -149,11 +149,16 @@ const TagSystem = ({ tagSystem, setTagSystem, agencyID}) => {
 
     const replaceTag = (tag) => {
         list[list.indexOf(selected)] = tag
-        if (active.includes(selected)) {
-            active[active.indexOf(selected)] = tag
+        try {
+            if (active.includes(selected)) {
+                active[active.indexOf(selected)] = tag
+            }
+            setData(tagSystem,list,active,agencyID)
+            setSelected("")
+        } catch (error) {
+            Sentry.captureException(error)
+            console.error("Error in replaceTag:", error)
         }
-        setData(tagSystem, list, active, agencyID)
-        setSelected("")
     }
 
     const addNewTag = (tag) => {
diff --git a/components/modals/RenameTagModal.jsx b/components/modals/RenameTagModal.jsx
index 0a8ca71..9305f0f 100644
--- a/components/modals/RenameTagModal.jsx
+++ b/components/modals/RenameTagModal.jsx
@@ -1,5 +1,6 @@
 import React, { useState } from 'react'
 import { IoClose } from "react-icons/io5"
+import * as Sentry from '@sentry/react'
 
 const RenameTagModal = ({ replaceTag, selected, list, setRenameTagModal, addNewTag }) => {
     const lower = []
@@ -9,22 +10,35 @@ const RenameTagModal = ({ replaceTag, selected, list, setRenameTagModal, addNewT
     const [tag, setTag] = useState("")
 
     const handleChange = (e) => {
-        setTag(e.target.value)
+        try {
+            setTag(e.target.value)
+        } catch (error) {
+            console.error("Error in handleChange:", error)
+        }
     }
 
     const handleAddNewTag = (e) => {
         e.preventDefault()
-        if (tag.length != 0 && tag.length <= 20) {
-            addNewTag(tag)
-            setRenameTagModal(false)
+        try {
+            if (tag.length != 0 && tag.length <= 20) {
+                addNewTag(tag)
+                setRenameTagModal(false)
+            }
+        } catch (error) {
+            console.error("Error in handleAddNewTag:", error)
         }
     }
 
     const handleReplaceTag = (e) => {
         e.preventDefault()
-        if (!lower.includes(tag.toLowerCase()) && tag.length != 0 && tag.length <= 20) {
-            replaceTag(tag)
-            setRenameTagModal(false)
+        try {
+            if (!lower.includes(tag.toLowerCase()) && tag.length != 0 && tag.length <= 20) {
+                replaceTag(tag)
+                setRenameTagModal(false)
+            }
+        } catch (error) {
+            Sentry.captureException(error)
+            console.error("Error in handleReplaceTag:", error)
         }
     }
 

From 9e1a340abab3e9bc118d893cda141d25fa15c4b8 Mon Sep 17 00:00:00 2001
From: Luke C Hartman <luke@lukecarlhartman.com>
Date: Tue, 22 Oct 2024 12:17:45 -0700
Subject: [PATCH 4/7] rename tag length 25 chars

---
 components/modals/RenameTagModal.jsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/components/modals/RenameTagModal.jsx b/components/modals/RenameTagModal.jsx
index 9305f0f..ac20e19 100644
--- a/components/modals/RenameTagModal.jsx
+++ b/components/modals/RenameTagModal.jsx
@@ -32,7 +32,7 @@ const RenameTagModal = ({ replaceTag, selected, list, setRenameTagModal, addNewT
     const handleReplaceTag = (e) => {
         e.preventDefault()
         try {
-            if (!lower.includes(tag.toLowerCase()) && tag.length != 0 && tag.length <= 20) {
+            if (!lower.includes(tag.toLowerCase()) && tag.length != 0 && tag.length <= 25) {
                 replaceTag(tag)
                 setRenameTagModal(false)
             }

From 9d48a987f3892bc76ce2cbd6fe0deff9a7c732b7 Mon Sep 17 00:00:00 2001
From: Tomin <tchazhikat@gmail.com>
Date: Tue, 22 Oct 2024 15:59:00 -0500
Subject: [PATCH 5/7] Added lazy load to user list dashboard

---
 components/Users.jsx | 43 ++++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 42 insertions(+), 1 deletion(-)

diff --git a/components/Users.jsx b/components/Users.jsx
index db0c446..91c9a5c 100644
--- a/components/Users.jsx
+++ b/components/Users.jsx
@@ -93,6 +93,45 @@ const sortByJoinedDate = (users) => {
 
 const Users = () => {
 	// Initialize authentication context
+	const [hasMore, setHasMore] = useState(true);
+	const [page, setPage] = useState(0);
+	const pageSize = 10;
+
+	const fetchMoreData = async () => {
+		if (isLoading) return;
+
+		setIsLoading(true);
+		try {
+			const mobileUsersQuerySnapshot = await getDocs(
+				query(
+					collection(db, 'mobileUsers'),
+					limit(pageSize),
+					startAfter(page * pageSize)
+				)
+			);
+
+			const newUsers = await Promise.all(
+				mobileUsersQuerySnapshot.docs.map(async (doc) => {
+					const userData = doc.data();
+					userData.mobileUserId = doc.id;
+					userData.disabled = await fetchUserDetails(doc.id);
+					userData.joined = getJoinedDate(userData, null);
+					return userData;
+				})
+			);
+
+			if (newUsers.length < pageSize) {
+				setHasMore(false);
+			}
+
+			setLoadedMobileUsers((prevUsers) => [...prevUsers, ...newUsers]);
+			setPage((prevPage) => prevPage + 1);
+		} catch (error) {
+			console.error('Failed to fetch more user data:', error);
+		} finally {
+			setIsLoading(false);
+		}
+	};
 	const {
 		user,
 		addAdminRole,
@@ -848,7 +887,9 @@ const Users = () => {
 							className="overflow-x-auto"
 							dataLength={endIndex}
 							inverse={false}
-							scrollableTarget="scrollableDiv">
+							scrollableTarget="scrollableDiv"
+							next={fetchMoreData}
+							hasMore={hasMore}>
 							<table className="min-w-full bg-white rounded-xl p-1">
 								<thead className="border-b dark:border-indigo-100 bg-slate-100">
 									<tr>

From 8458edb82f6d8d13a4368c988da3463607d611c1 Mon Sep 17 00:00:00 2001
From: Luke C Hartman <luke@lukecarlhartman.com>
Date: Tue, 22 Oct 2024 15:24:58 -0700
Subject: [PATCH 6/7] Revert "Added lazy load to user list dashboard"

---
 components/Users.jsx | 43 +------------------------------------------
 1 file changed, 1 insertion(+), 42 deletions(-)

diff --git a/components/Users.jsx b/components/Users.jsx
index 91c9a5c..db0c446 100644
--- a/components/Users.jsx
+++ b/components/Users.jsx
@@ -93,45 +93,6 @@ const sortByJoinedDate = (users) => {
 
 const Users = () => {
 	// Initialize authentication context
-	const [hasMore, setHasMore] = useState(true);
-	const [page, setPage] = useState(0);
-	const pageSize = 10;
-
-	const fetchMoreData = async () => {
-		if (isLoading) return;
-
-		setIsLoading(true);
-		try {
-			const mobileUsersQuerySnapshot = await getDocs(
-				query(
-					collection(db, 'mobileUsers'),
-					limit(pageSize),
-					startAfter(page * pageSize)
-				)
-			);
-
-			const newUsers = await Promise.all(
-				mobileUsersQuerySnapshot.docs.map(async (doc) => {
-					const userData = doc.data();
-					userData.mobileUserId = doc.id;
-					userData.disabled = await fetchUserDetails(doc.id);
-					userData.joined = getJoinedDate(userData, null);
-					return userData;
-				})
-			);
-
-			if (newUsers.length < pageSize) {
-				setHasMore(false);
-			}
-
-			setLoadedMobileUsers((prevUsers) => [...prevUsers, ...newUsers]);
-			setPage((prevPage) => prevPage + 1);
-		} catch (error) {
-			console.error('Failed to fetch more user data:', error);
-		} finally {
-			setIsLoading(false);
-		}
-	};
 	const {
 		user,
 		addAdminRole,
@@ -887,9 +848,7 @@ const Users = () => {
 							className="overflow-x-auto"
 							dataLength={endIndex}
 							inverse={false}
-							scrollableTarget="scrollableDiv"
-							next={fetchMoreData}
-							hasMore={hasMore}>
+							scrollableTarget="scrollableDiv">
 							<table className="min-w-full bg-white rounded-xl p-1">
 								<thead className="border-b dark:border-indigo-100 bg-slate-100">
 									<tr>

From f55a8de88418d795864d97363213ff5cbff7c053 Mon Sep 17 00:00:00 2001
From: Ethan Lanting <ethanlanting@gmail.com>
Date: Thu, 24 Oct 2024 16:15:23 -0500
Subject: [PATCH 7/7] refactor CSV conversion to include user email in the
 report

---
 components/ReportsSection.jsx | 76 +++++++++++++++++++++++------------
 1 file changed, 50 insertions(+), 26 deletions(-)

diff --git a/components/ReportsSection.jsx b/components/ReportsSection.jsx
index 4a4204a..dee01af 100644
--- a/components/ReportsSection.jsx
+++ b/components/ReportsSection.jsx
@@ -433,14 +433,33 @@ const ReportsSection = ({
 		return Array.from(headersSet) // Convert Set to Array
 	}
 
+	const userIDToEmail = async (userID) => {
+		const docRef = doc(db, 'mobileUsers', userID)
+		const docSnap = await getDoc(docRef)
+		if (docSnap.exists()) {
+			return docSnap.data().email
+		} else {
+			console.log('No such document!')
+			return ''
+		}
+	}
+
 	// Convert JSON array to CSV
-	const convertToCSV = (jsonArray) => {
+	const convertToCSV = async (jsonArray) => {
 		const headers = extractHeaders(jsonArray)
 
+		// Ensure 'images' is the last header. Multiple images mess up the CSV format
+		const filteredHeaders = headers.filter((header) => header !== 'images')
+		// Add 'userEmail' column after 'userID'
+		const extendedHeaders = filteredHeaders.flatMap((header) =>
+			header === 'userID' ? ['userID', 'userEmail'] : [header],
+		)
+		const finalHeaders = [...extendedHeaders, 'images']
+
 		const csvRows = []
 
 		// Add headers row
-		csvRows.push(headers.join(','))
+		csvRows.push(finalHeaders.join(','))
 
 		// Function to convert `createdDate` to ISO 8601 format
 		const formatDateToISO = (createdDate) => {
@@ -455,40 +474,45 @@ const ReportsSection = ({
 		}
 
 		// Loop through each report and convert to CSV row
-		jsonArray.forEach((report) => {
-			const row = headers.map((header) => {
-				let value
+		for (const report of jsonArray) {
+			const row = await Promise.all(
+				finalHeaders.map(async (header) => {
+					let value
+
+					// Special case for the `createdDate`
+					if (header === 'createdDate') {
+						value = formatDateToISO(report.createdDate)
+					} else if (header === 'userEmail' && report.userID) {
+						// Fetch the email for the corresponding userID
+						value = await userIDToEmail(report.userID)
+					} else {
+						const keys = header.split('.')
+						value = report
+						keys.forEach((key) => {
+							value = value[key] !== undefined ? value[key] : '' // Safely access nested values
+						})
 
-				// Special case for the `createdDate`
-				if (header === 'createdDate') {
-					value = formatDateToISO(report.createdDate)
-				} else {
-					const keys = header.split('.')
-					value = report
-					keys.forEach((key) => {
-						value = value[key] !== undefined ? value[key] : '' // Safely access nested values
-					})
-
-					// Handle commas and newlines in CSV fields
-					if (typeof value === 'string') {
-						value = value.replace(/"/g, '""') // Escape double quotes
-						if (value.includes(',') || value.includes('\n')) {
-							value = `"${value}"` // Wrap in double quotes if necessary
+						// Handle commas and newlines in CSV fields
+						if (typeof value === 'string') {
+							value = value.replace(/"/g, '""') // Escape double quotes
+							if (value.includes(',') || value.includes('\n')) {
+								value = `"${value}"` // Wrap in double quotes if necessary
+							}
 						}
 					}
-				}
 
-				return value
-			})
+					return value
+				}),
+			)
 			csvRows.push(row.join(','))
-		})
+		}
 
 		return csvRows.join('\n')
 	}
 
 	// Trigger download of CSV file
-	const downloadCSV = () => {
-		const csvData = convertToCSV(reports)
+	const downloadCSV = async () => {
+		const csvData = await convertToCSV(reports)
 		const blob = new Blob([csvData], { type: 'text/csv;charset=utf-8;' })
 		const url = URL.createObjectURL(blob)