Skip to content

Commit c0f7f87

Browse files
committed
Fix service filter in sidebar
1 parent 62909bd commit c0f7f87

File tree

3 files changed

+38
-4
lines changed

3 files changed

+38
-4
lines changed

src/components/Admin/common/RepositorySidebarOverlay/index.tsx

Lines changed: 19 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,13 @@ export const RepositorySidebarOverlay = ({
7171
const insightIdToOpenSuggestion = useAdminSelector(
7272
(state) => state.repository.issues.insightIdToOpenSuggestion
7373
);
74-
const { resetInsights, resetAssets, resetGlobalErrors } = useStore.getState();
74+
const {
75+
resetInsights,
76+
resetAssets,
77+
resetGlobalErrors,
78+
setSelectedServices,
79+
clearInsightsFilters
80+
} = useStore.getState();
7581
const dispatch = useAdminDispatch();
7682
const [history, setHistory] = useState(
7783
() => new History<RepositorySidebarHistoryState>([])
@@ -212,6 +218,18 @@ export const RepositorySidebarOverlay = ({
212218
dispatch
213219
]);
214220

221+
// Set selected services on query change
222+
useEffect(() => {
223+
setSelectedServices(sidebarQuery?.query?.services ?? []);
224+
}, [sidebarQuery?.query?.services, setSelectedServices]);
225+
226+
// Clear insights filters on sidebar close
227+
useEffect(() => {
228+
if (!isSidebarOpen) {
229+
clearInsightsFilters();
230+
}
231+
}, [isSidebarOpen, clearInsightsFilters]);
232+
215233
const handleSidebarTransitionStart = () => {
216234
setIsSidebarTransitioning(true);
217235
};

src/components/Insights/hooks/useInsightsData.ts

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import { useCallback, useEffect, useMemo } from "react";
22
import { getFeatureFlagValue } from "../../../featureFlags";
3+
import { platform } from "../../../platform";
34
import {
45
useGetInsightsQuery,
56
useGetIssuesQuery,
@@ -50,7 +51,12 @@ export const useInsightsData = () => {
5051
const spanCodeObjectId = scope?.span?.spanCodeObjectId ?? null;
5152

5253
const filteredServices = useMemo(
53-
() => (spanCodeObjectId ? [] : selectedServices ?? []),
54+
() =>
55+
spanCodeObjectId
56+
? platform === "Web"
57+
? selectedServices ?? []
58+
: []
59+
: selectedServices ?? [],
5460
[selectedServices, spanCodeObjectId]
5561
);
5662
const filteredInsightTypes = spanCodeObjectId
@@ -150,7 +156,11 @@ export const useInsightsData = () => {
150156
showDismissed,
151157
scopedSpanCodeObjectId: spanCodeObjectId ?? undefined,
152158
insightTypes: filteredInsightTypes,
153-
services: spanCodeObjectId ? [] : filteredServices,
159+
services: spanCodeObjectId
160+
? platform === "Web"
161+
? filteredServices
162+
: []
163+
: filteredServices,
154164
...(isCriticalityLevelsFilterEnabled
155165
? { criticalityFilter: filteredCriticalityLevels }
156166
: {}),

src/components/Insights/hooks/useInsightsStats.ts

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import { useCallback, useMemo } from "react";
22
import { getFeatureFlagValue } from "../../../featureFlags";
3+
import { platform } from "../../../platform";
34
import { useGetInsightsStatsQuery } from "../../../redux/services/digma";
45
import { useConfigSelector } from "../../../store/config/useConfigSelector";
56
import { useInsightsSelector } from "../../../store/insights/useInsightsSelector";
@@ -37,7 +38,12 @@ export const useInsightsStats = ({
3738
? filteredInsightTypesInSpanScope
3839
: filteredInsightTypesInGlobalScope;
3940
const filteredServices = useMemo(
40-
() => (spanCodeObjectId ? [] : services ?? []),
41+
() =>
42+
spanCodeObjectId
43+
? platform === "Web"
44+
? services ?? []
45+
: []
46+
: services ?? [],
4147
[services, spanCodeObjectId]
4248
);
4349

0 commit comments

Comments
 (0)