Skip to content

Commit d2c89da

Browse files
committed
Merge branch 'app_details_env_filter' into env_filter_app_overview_deployment_metrics
2 parents 1043f90 + 122ea62 commit d2c89da

File tree

3 files changed

+14
-12
lines changed

3 files changed

+14
-12
lines changed

src/components/ApplicationGroup/AppGroupAppFilter.components.tsx

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -16,16 +16,21 @@ import { ConditionalWrap } from '@devtron-labs/devtron-fe-common-lib'
1616
import Tippy from '@tippyjs/react'
1717

1818
export const ValueContainer = (props): JSX.Element => {
19-
const { appListOptions, selectedAppList, selectedFilterTab, selectedGroupFilter, filterParentType }: AppGroupAppFilterContextType =
20-
useAppGroupAppFilterContext()
19+
const {
20+
appListOptions,
21+
selectedAppList,
22+
selectedFilterTab,
23+
selectedGroupFilter,
24+
filterParentType,
25+
}: AppGroupAppFilterContextType = useAppGroupAppFilterContext()
2126
let selectorText,
2227
selectedAppsLength = props.getValue().length
2328
if (selectedFilterTab === AppFilterTabs.GROUP_FILTER && selectedGroupFilter[0]) {
2429
selectorText = selectedGroupFilter[0]?.label
2530
} else {
2631
selectorText = `${selectedAppList.length > 0 ? selectedAppList.length : appListOptions.length}/${
2732
appListOptions.length
28-
} ${filterParentType===FilterParentType.env? 'Applications': 'Environments'}`
33+
} ${filterParentType === FilterParentType.env ? 'Applications' : 'Environments'}`
2934
}
3035
return (
3136
<components.ValueContainer {...props}>
@@ -144,7 +149,7 @@ export const MenuList = (props: any): JSX.Element => {
144149
openCreateGroup,
145150
selectedGroupFilter,
146151
setSelectedGroupFilter,
147-
filterParentType
152+
filterParentType,
148153
}: AppGroupAppFilterContextType = useAppGroupAppFilterContext()
149154
const clearSelection = (): void => {
150155
setSelectedAppList([])
@@ -153,7 +158,7 @@ export const MenuList = (props: any): JSX.Element => {
153158
const onTabChange = (e): void => {
154159
setSelectedFilterTab(e.currentTarget.dataset.selectedTab)
155160
}
156-
const selectedType = filterParentType===FilterParentType.env? 'applications': 'environments'
161+
const selectedType = filterParentType === FilterParentType.env ? 'applications' : 'environments'
157162
return (
158163
<components.MenuList {...props}>
159164
<div className="dc__position-sticky dc__top-0 bcn-0">
@@ -208,8 +213,8 @@ export const MenuList = (props: any): JSX.Element => {
208213
<InfoIcon className="icon-dim-20 mr-4 mw-18 cursor fcn-6 mb-4" />
209214
<div className="fs-13 fw-6 cn-9 mb-4">No saved filters</div>
210215
<div className="fs-12 fw-4 cn-7 dc__align-center ">
211-
To save a filter, select some {selectedType} from All {selectedType} and click on ‘Save selection as
212-
filter’
216+
To save a filter, select some {selectedType} from All {selectedType} and click on ‘Save
217+
selection as filter’
213218
</div>
214219
</div>
215220
)}

src/components/app/details/cdDetails/CDDetails.tsx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -156,8 +156,8 @@ export default function CDDetails({filteredEnvIds}:{filteredEnvIds: string}) {
156156
const filteredEnvMap = filteredEnvIds?.split(',').reduce((agg, curr) => agg.set(curr, true), new Map())
157157
let _selectedEnvironment = (result[0]['value']?.result || [])
158158
.filter((env) => {
159-
console.log(filteredEnvMap?.get(env.environmentId))
160-
return !filteredEnvMap || filteredEnvMap.get(env.environmentId)})
159+
return !filteredEnvMap || filteredEnvMap.get(env.environmentId)
160+
})
161161
.find((envData) => {
162162
return +envId === envData.environmentId
163163
})
@@ -185,7 +185,6 @@ export default function CDDetails({filteredEnvIds}:{filteredEnvIds: string}) {
185185
const filteredEnvMap = filteredEnvIds?.split(',').reduce((agg, curr) => agg.set(+curr, true), new Map())
186186
const envOptions: CICDSidebarFilterOptionType[] = (result[0]['value']?.result || [])
187187
.filter((env) => {
188-
console.log(filteredEnvMap?.get(env.environmentId))
189188
return !filteredEnvMap || filteredEnvMap.get(env.environmentId)})
190189
.map((envData) => {
191190
if (envData.environmentId === +envId) {

src/components/app/details/main.tsx

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -94,8 +94,6 @@ export default function AppDetailsPage({ isV2 }: AppDetailsProps) {
9494
setSelectedAppList([])
9595
setAppListLoading(true)
9696
const { result } = await getAppOtherEnvironmentMin(appId)
97-
console.log(result)
98-
//setAppGroupListData(result)
9997
if (result?.length) {
10098
setAppListOptions(
10199
result

0 commit comments

Comments
 (0)