Skip to content

Commit 718edf0

Browse files
krithika369Krithika Sundararajan
andauthored
Fix Configure Alerts button not showing up (#210)
Co-authored-by: Krithika Sundararajan <[email protected]>
1 parent e396d8c commit 718edf0

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

ui/src/router/alerts/details/RouterAlertDetails.js

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import React, { Fragment, useMemo, useRef } from "react";
1+
import React, { Fragment, useRef } from "react";
22
import {
33
EuiFlexGroup,
44
EuiFlexItem,
@@ -18,13 +18,14 @@ import { supportedAlerts } from "../config";
1818

1919
export const RouterAlertDetails = ({ alertsData, routerStatus, ...props }) => {
2020
const alertDetailsRef = useRef();
21-
const status = useMemo(() => Status.fromValue(routerStatus), [routerStatus]);
2221

2322
return (
2423
<div ref={alertDetailsRef}>
25-
{[Status.UNDEPLOYED, Status.PENDING].includes(status) && (
24+
{[Status.UNDEPLOYED, Status.PENDING].includes(routerStatus) && (
2625
<OverlayMask parentRef={alertDetailsRef} opacity={0.4}>
27-
{status === Status.PENDING && <EuiLoadingChart size="xl" mono />}
26+
{routerStatus === Status.PENDING && (
27+
<EuiLoadingChart size="xl" mono />
28+
)}
2829
</OverlayMask>
2930
)}
3031
<EuiFlexGroup alignItems="baseline">
@@ -41,7 +42,7 @@ export const RouterAlertDetails = ({ alertsData, routerStatus, ...props }) => {
4142
<EuiButton
4243
size="s"
4344
onClick={() => props.navigate("./edit")}
44-
disabled={status !== Status.DEPLOYED}>
45+
disabled={routerStatus !== Status.DEPLOYED}>
4546
Configure Alerts
4647
</EuiButton>
4748
</EuiFlexItem>

0 commit comments

Comments
 (0)