From 424ca142c195bfae1852f0e93433df02305e90dc Mon Sep 17 00:00:00 2001
From: Timothy Carambat <rambat1010@gmail.com>
Date: Thu, 22 Feb 2024 09:15:27 -0800
Subject: [PATCH] Fix default role visibility permissions (#776)

---
 frontend/src/App.jsx                                       | 2 +-
 frontend/src/components/Sidebar/ActiveWorkspaces/index.jsx | 6 ++----
 2 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/frontend/src/App.jsx b/frontend/src/App.jsx
index 7633af2c..86f6eb08 100644
--- a/frontend/src/App.jsx
+++ b/frontend/src/App.jsx
@@ -59,7 +59,7 @@ export default function App() {
               <Route path="/login" element={<Login />} />
               <Route
                 path="/workspace/:slug/settings/:tab"
-                element={<PrivateRoute Component={WorkspaceSettings} />}
+                element={<ManagerRoute Component={WorkspaceSettings} />}
               />
               <Route
                 path="/workspace/:slug"
diff --git a/frontend/src/components/Sidebar/ActiveWorkspaces/index.jsx b/frontend/src/components/Sidebar/ActiveWorkspaces/index.jsx
index 9ee9c40c..9ee6a687 100644
--- a/frontend/src/components/Sidebar/ActiveWorkspaces/index.jsx
+++ b/frontend/src/components/Sidebar/ActiveWorkspaces/index.jsx
@@ -114,10 +114,8 @@ export default function ActiveWorkspaces() {
                         : truncate(workspace.name, 20)}
                     </p>
                   </div>
-                  {isActive ||
-                  isHovered ||
-                  gearHover[workspace.id] ||
-                  user?.role === "default" ? (
+                  {(isActive || isHovered || gearHover[workspace.id]) &&
+                  user?.role !== "default" ? (
                     <div className="flex items-center gap-x-2">
                       <button
                         type="button"