Merge pull request #927 from bluewave-labs/fix/maintenance-access-rights

Allow demo user to create maintenance windows
bluewave-labs · Oct 11, 2024 · e052adf · e052adf
2 parents 7a8d909 + 34e2ecd
commit e052adf
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 14 deletions.
diff --git a/Client/src/Pages/Maintenance/index.jsx b/Client/src/Pages/Maintenance/index.jsx
@@ -93,7 +93,7 @@ const Maintenance = ({ isAdmin }) => {
           />
         </Stack>
       )}
-      {isAdmin && maintenanceWindows.length === 0 && (
+      {maintenanceWindows.length === 0 && (
         <Fallback
           title="maintenance window"
           checks={[
@@ -102,7 +102,7 @@ const Maintenance = ({ isAdmin }) => {
             "Stop sending alerts in maintenance windows",
           ]}
           link="/maintenance/create"
-          isAdmin={isAdmin}
+          isAdmin={true}
         />
       )}
     </Box>

diff --git a/Client/src/Pages/Monitors/Home/index.jsx b/Client/src/Pages/Monitors/Home/index.jsx
@@ -59,18 +59,19 @@ const Monitors = ({ isAdmin }) => {
               mt={theme.spacing(5)}
             >
               <Greeting type="uptime" />
-              {monitorState?.monitorsSummary?.monitors?.length !== 0 && (
-                <Button
-                  variant="contained"
-                  color="primary"
-                  onClick={() => {
-                    navigate("/monitors/create");
-                  }}
-                  sx={{ fontWeight: 500 }}
-                >
-                  Create monitor
-                </Button>
-              )}
+              {isAdmin &&
+                monitorState?.monitorsSummary?.monitors?.length !== 0 && (
+                  <Button
+                    variant="contained"
+                    color="primary"
+                    onClick={() => {
+                      navigate("/monitors/create");
+                    }}
+                    sx={{ fontWeight: 500 }}
+                  >
+                    Create monitor
+                  </Button>
+                )}
             </Stack>
           </Box>
           {isAdmin && monitorState?.monitorsSummary?.monitors?.length === 0 && (