paulononato/backstage
1
0
Fork 0
Code Issues Pull Requests Actions 44 Packages Projects Releases Wiki Activity

memoize alerts, not context

Browse Source
This commit is contained in:
Ryan Vazquez
2021-01-27 15:32:29 -05:00
parent 027ee1a7cf
commit 8180fb76d8
1 changed files with 12 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
plugins/cost-insights/src/components/CostInsightsPage/CostInsightsPage.tsx
+12 -4
View File
@@ -78,10 +78,18 @@ export const CostInsightsPage = () => {
const { pageFilters, setPageFilters } = useFilters(p => p);
const snoozed = useMemo(() => alerts.alerts.filter(isSnoozed), [alerts]);
const accepted = useMemo(() => alerts.alerts.filter(isAccepted), [alerts]);
const dismissed = useMemo(() => alerts.alerts.filter(isDismissed), [alerts]);
const activeAlerts = useMemo(() => alerts.alerts.filter(isActive), [alerts]);
const snoozed = useMemo(() => alerts.alerts.filter(isSnoozed), [
alerts.alerts,
]);
const accepted = useMemo(() => alerts.alerts.filter(isAccepted), [
alerts.alerts,
]);
const dismissed = useMemo(() => alerts.alerts.filter(isDismissed), [
alerts.alerts,
]);
const activeAlerts = useMemo(() => alerts.alerts.filter(isActive), [
alerts.alerts,
]);
const isActionItemsDisplayed = !!activeAlerts.length;
const isAlertInsightsDisplayed = !!alerts.alerts.length;