From 71187d43be8b67115dc1ac1587e5ffc9d445365c Mon Sep 17 00:00:00 2001
From: Om Londhe <61611599+omlondhe@users.noreply.github.com>
Date: Sun, 7 May 2023 17:06:35 +0530
Subject: [PATCH] fix: removed unnecessary useState  (#6580)

* Removed unnecessary useState

* Update search.js

Removed `_` as per my conversation with **Sir Eddie Jaoude**.
---
 components/navbar/Navbar.js | 17 ++++++++---------
 pages/events.js             |  8 +++-----
 pages/search.js             |  4 ++--
 3 files changed, 13 insertions(+), 16 deletions(-)

diff --git a/components/navbar/Navbar.js b/components/navbar/Navbar.js
index 2373fe1ca19..46786b443ec 100644
--- a/components/navbar/Navbar.js
+++ b/components/navbar/Navbar.js
@@ -14,20 +14,19 @@ const FaRegMoon = getIcon("FaRegMoon");
 const FaSun = getIcon("FaSun");
 
 export default function Navbar() {
-  const { systemTheme, theme, setTheme } = useTheme();
+  const [isOpen, setIsOpen] = useState(false);
   const [mounted, setMounted] = useState(false);
 
+  const router = useRouter();
+  const navConRef = useRef();
+  const { data: session } = useSession();
+  const { systemTheme, theme, setTheme } = useTheme();
+  const getLink = (path) => `${router.basePath}${path}`;
+
   useEffect(() => {
     setMounted(true);
   }, []);
 
-  const { data: session } = useSession();
-  const [isOpen, setIsOpen] = useState(false);
-
-  const router = useRouter();
-  const getLink = (path) => `${router.basePath}${path}`;
-  const navConRef = useRef();
-
   const renderThemeChanger = () => {
     if (!mounted) {
       return null;
@@ -38,7 +37,7 @@ export default function Navbar() {
     if (currentTheme === "dark") {
       return (
         <button
-          className="p-2" 
+          className="p-2"
           onClick={() => setTheme("light")}
           aria-label="Toggle Theme"
         >
diff --git a/pages/events.js b/pages/events.js
index c677bcf7ca8..4b121d74bb4 100644
--- a/pages/events.js
+++ b/pages/events.js
@@ -53,7 +53,7 @@ export default function Events({ events }) {
       event.date.cfpClose ? new Date(event.date.cfpClose) > new Date() : false
     ),
   };
-  const filters = [
+  const tabFilters = [
     {
       title: "Show all",
       description: "List all events with no filters",
@@ -84,7 +84,6 @@ export default function Events({ events }) {
     },
   ];
 
-  const [tabs, setTabs] = useState(filters);
   const [eventType, setEventType] = useState("all");
 
   return (
@@ -106,15 +105,14 @@ export default function Events({ events }) {
           </Badge>
         </div>
         <EventTabs
-          tabs={tabs}
+          tabs={tabFilters}
           eventType={eventType}
           setEventType={setEventType}
         />
         <h2 className="text-md md:text-2xl text-lg text-primary-high font-bold md:mb-6 mb-3">
-            {filters.find((filter) => filter.key === eventType).description}
+          {tabFilters.find((filter) => filter.key === eventType).description}
         </h2>
         <ul role="list" className="divide-y divide-primary-low mt-6">
-
           {categorisedEvents[eventType]?.map((event) => (
             <EventCard
               event={event}
diff --git a/pages/search.js b/pages/search.js
index bcf6a146a97..16cfba16339 100755
--- a/pages/search.js
+++ b/pages/search.js
@@ -10,7 +10,7 @@ import Badge from "@components/Badge";
 import logger from "@config/logger";
 import Input from "@components/form/Input";
 
-export async function getServerSideProps(context) {
+export async function getServerSideProps() {
   let data = {
     users: [],
     tags: [],
@@ -40,8 +40,8 @@ export default function Search({ data }) {
   let { users, tags } = data;
   const router = useRouter();
   const { username, keyword } = router.query;
-  const [filteredUsers, setFilteredUsers] = useState([]);
   const [notFound, setNotFound] = useState();
+  const [filteredUsers, setFilteredUsers] = useState([]);
   const [inputValue, setInputValue] = useState(username || keyword || "");
 
   let results = [];