From 198c40132fee6793130fabd071bb52d89580711f Mon Sep 17 00:00:00 2001
From: Kurt Hutten <kurt@kurthutten.com>
Date: Tue, 10 Nov 2020 20:37:04 +1100
Subject: [PATCH] Fix new button,

also redirects if user is not signed in
---
 web/src/layouts/MainLayout/MainLayout.js   | 9 ++++++---
 web/src/pages/NewPart2Page/NewPart2Page.js | 7 ++++++-
 2 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/web/src/layouts/MainLayout/MainLayout.js b/web/src/layouts/MainLayout/MainLayout.js
index 33a8215..1f9fbf7 100644
--- a/web/src/layouts/MainLayout/MainLayout.js
+++ b/web/src/layouts/MainLayout/MainLayout.js
@@ -72,9 +72,12 @@ const MainLayout = ({ children}) => {
           </ul>
           <ul className="flex items-center">
             <li className={getActiveClasses("mr-8 h-10 w-10 rounded-full border-2 border-gray-700 flex items-center justify-center", {'border-indigo-300': currentUser})}>
-              <button className="h-full w-full" onClick={() => currentUser && routes.newPart2({userName: data?.user?.userName})}>
-              <Svg name="plus" className={getActiveClasses("text-gray-700 w-full h-full",{'text-indigo-300': currentUser})} />
-              </button>
+              {isAuthenticated && data?.user?.userName ?
+                <Link className="h-full w-full" to={routes.newPart2({userName: data?.user?.userName})}>
+                  <Svg name="plus" className="text-indigo-300 w-full h-full" />
+                </Link>:
+                <Svg name="plus" className="text-gray-700 w-full h-full" />
+              }
             </li>
             <li className="h-10 w-10 border-1 rounded-full border-indigo-300 text-indigo-200">
             <div aria-describedby={popoverId} onMouseOver={togglePopover}>
diff --git a/web/src/pages/NewPart2Page/NewPart2Page.js b/web/src/pages/NewPart2Page/NewPart2Page.js
index bd316a9..c1cb4e4 100644
--- a/web/src/pages/NewPart2Page/NewPart2Page.js
+++ b/web/src/pages/NewPart2Page/NewPart2Page.js
@@ -1,11 +1,16 @@
+import { useEffect } from 'react'
 import { useAuth } from '@redwoodjs/auth'
+import { navigate, routes } from '@redwoodjs/router'
 
 import MainLayout from 'src/layouts/MainLayout'
 import Part2Cell from 'src/components/Part2Cell'
 
 
 const NewPart2Page = ({userName}) => {
-  const { currentUser } = useAuth()
+  const { isAuthenticated, currentUser } = useAuth()
+  useEffect(() => {!isAuthenticated &&
+    navigate(routes.home())},
+  [currentUser])
   return (
     <MainLayout>
       <Part2Cell