From e546eaa0702ab8a7ffd4b72861c2e6a92dc7e600 Mon Sep 17 00:00:00 2001
From: Desmond Grealy <grealydesmond@gmail.com>
Date: Mon, 2 Jan 2023 21:09:37 -0800
Subject: [PATCH] one more fix for header border-bottom. Also fixed some eslint
 whine

---
 website/src/components/Header/Header.tsx      | 14 ++++++++++----
 website/src/pages/api/new_task/[task_type].ts |  2 ++
 2 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/website/src/components/Header/Header.tsx b/website/src/components/Header/Header.tsx
index bdf011a4..2d0ef7cc 100644
--- a/website/src/components/Header/Header.tsx
+++ b/website/src/components/Header/Header.tsx
@@ -1,4 +1,4 @@
-import { Box, Button } from "@chakra-ui/react";
+import { Box, Button, useColorMode } from "@chakra-ui/react";
 import { Popover } from "@headlessui/react";
 import { AnimatePresence, motion } from "framer-motion";
 import Image from "next/image";
@@ -54,10 +54,16 @@ function AccountButton() {
 }
 
 export function Header(props) {
-  const borderClass = props.transparent ? "" : props.borderClass ?? "border-b border-gray-100";
+  const { colorMode } = useColorMode();
+  const borderClass = props.transparent
+    ? ""
+    : colorMode === "light"
+    ? "border-b border-gray-400"
+    : "border-b border-zinc-800";
+
   return (
-    <nav className="oa-basic-theme">
-      <Box className={`flex mx-auto max-w-7xl justify-between py-8 px-10 ${borderClass}`}>
+    <nav className={`oa-basic-theme ${borderClass}`}>
+      <Box className="flex mx-auto max-w-7xl justify-between py-8 px-10">
         <div className="relative z-10 flex items-center gap-16">
           <Link href="/" aria-label="Home" className="flex items-center">
             <Image src="/images/logos/logo.svg" className="mx-auto object-fill" width="50" height="50" alt="logo" />
diff --git a/website/src/pages/api/new_task/[task_type].ts b/website/src/pages/api/new_task/[task_type].ts
index 2a2a0215..bf2ec5c4 100644
--- a/website/src/pages/api/new_task/[task_type].ts
+++ b/website/src/pages/api/new_task/[task_type].ts
@@ -64,6 +64,8 @@ const handler = async (req, res) => {
     }),
   });
 
+  ackRes; // calling this only to get rid of the unused variable warning... not sure if anything is intended to be done with ackRes
+
   // Send the results to the client.
   res.status(200).json(registeredTask);
 };