Skip to content

Commit b46ea9f

Browse files
committed
Merge branch 'develop' of https://github.com/devtron-labs/dashboard into fix-ng-changes-develop
2 parents 3dc0ee3 + 301d009 commit b46ea9f

File tree

7 files changed

+18
-46
lines changed

7 files changed

+18
-46
lines changed

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
"private": true,
55
"homepage": "/dashboard",
66
"dependencies": {
7-
"@devtron-labs/devtron-fe-common-lib": "1.11.0-beta-7",
7+
"@devtron-labs/devtron-fe-common-lib": "1.11.0-pre-6",
88
"@esbuild-plugins/node-globals-polyfill": "0.2.3",
99
"@rjsf/core": "^5.13.3",
1010
"@rjsf/utils": "^5.13.3",

src/components/app/list/expandedRow/ExpandedRow.tsx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,10 @@ export class ExpandedRow extends Component<ExpandedRowProps> {
3636
<Link
3737
key={env.id}
3838
to={`${this.props.redirect(this.props.app, env.id)}`}
39-
className="app-list__row app-list__row--expanded"
39+
className={`app-list__row app-list__row--expanded ${!this.props.isArgoInstalled ? 'app-list__row--argo-not-installed' : ''}`}
4040
>
4141
<div className="app-list__cell--icon" />
42-
<div className="app-list__cell app-list__cell--name">
42+
<div className="app-list__cell app-list__cell--name dc__overflow-hidden">
4343
<svg className="app-status app-status--pseudo" preserveAspectRatio="none" viewBox="0 0 200 40">
4444
<line x1="0" y1="20" x2="100%" y2="20" stroke={color} strokeWidth="1" />
4545
<line x1="0" y1="15" x2="0" y2="25" stroke={color} strokeWidth="1" />
@@ -52,15 +52,15 @@ export class ExpandedRow extends Component<ExpandedRowProps> {
5252
)}
5353
<div className="app-list__cell app-list__cell--env">{env.name}</div>
5454
<div className="app-list__cell app-list__cell--cluster">
55-
<p className="dc__truncate-text">{env.clusterName}</p>
55+
<p className="dc__truncate-text m-0">{env.clusterName}</p>
5656
</div>
5757
<div className="app-list__cell app-list__cell--namespace">
58-
<p className="dc__truncate-text">{env.namespace}</p>
58+
<p className="dc__truncate-text m-0">{env.namespace}</p>
5959
</div>
6060
<div className="app-list__cell app-list__cell--time">
6161
{env.lastDeployedTime && (
6262
<Tippy className="default-tt" arrow placement="top" content={env.lastDeployedTime}>
63-
<p className="dc__truncate-text m-0">{handleUTCTime(env.lastDeployedTime, true)}</p>
63+
<p className="dc__truncate-text m-0">{handleUTCTime(env.lastDeployedTime, true)}</p>
6464
</Tippy>
6565
)}
6666
</div>

src/components/common/DynamicTabs/types.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -75,12 +75,6 @@ export interface TimerType {
7575
format?: (start: Dayjs, now: Dayjs) => string
7676
}
7777

78-
export type ParsedTabsDataV1 = {
79-
key: string
80-
data: DynamicTabType[]
81-
version: 'v1'
82-
}
83-
8478
export type ParsedTabsData = {
8579
data: Record<string, DynamicTabType[]>
8680
version: typeof TAB_DATA_VERSION

src/components/common/DynamicTabs/useTabs.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ import { DynamicTabType, InitTabType, noop } from '@devtron-labs/devtron-fe-comm
2121

2222
import { FALLBACK_TAB, TAB_DATA_LOCAL_STORAGE_KEY, TAB_DATA_VERSION } from './constants'
2323
import { AddTabParamsType, ParsedTabsData, PopulateTabDataPropsType, UseTabsReturnType } from './types'
24-
import { convertV1TabsDataToV2 } from './utils'
2524

2625
export function useTabs(persistenceKey: string, fallbackTabIndex = FALLBACK_TAB): UseTabsReturnType {
2726
const [tabs, setTabs] = useState<DynamicTabType[]>([])
@@ -107,7 +106,7 @@ export function useTabs(persistenceKey: string, fallbackTabIndex = FALLBACK_TAB)
107106
} else {
108107
const persistedTabsData = getTabDataFromLocalStorage()
109108
try {
110-
_parsedTabsData = convertV1TabsDataToV2(JSON.parse(persistedTabsData))
109+
_parsedTabsData = JSON.parse(persistedTabsData)
111110
} catch {
112111
noop()
113112
}
@@ -177,7 +176,7 @@ export function useTabs(persistenceKey: string, fallbackTabIndex = FALLBACK_TAB)
177176
if (!reInit) {
178177
const persistedTabsData = getTabDataFromLocalStorage()
179178
try {
180-
parsedTabsData = convertV1TabsDataToV2(JSON.parse(persistedTabsData))
179+
parsedTabsData = JSON.parse(persistedTabsData)
181180
_tabs = parsedTabsData ? parsedTabsData.data[persistenceKey] ?? [] : prevTabs
182181
} catch {
183182
_tabs = prevTabs

src/components/common/DynamicTabs/utils.ts

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,7 @@
1717
import { Dayjs } from 'dayjs'
1818

1919
import { MARK_AS_STALE_DATA_CUT_OFF_MINS } from '../../ResourceBrowser/Constants'
20-
import { TAB_DATA_VERSION } from './constants'
21-
import { DynamicTabsVariantType, ParsedTabsData, ParsedTabsDataV1 } from './types'
20+
import { DynamicTabsVariantType } from './types'
2221

2322
export const checkIfDataIsStale = (start: Dayjs, now: Dayjs): boolean =>
2423
now.diff(start, 'minutes') > MARK_AS_STALE_DATA_CUT_OFF_MINS
@@ -35,14 +34,3 @@ export const getClassNameForVariant = (variant: DynamicTabsVariantType) => {
3534
return ''
3635
}
3736
}
38-
39-
export const convertV1TabsDataToV2 = (tabsData: ParsedTabsDataV1 | ParsedTabsData): ParsedTabsData => {
40-
if (tabsData.version === TAB_DATA_VERSION) {
41-
return tabsData
42-
}
43-
44-
return {
45-
data: { [tabsData.key]: tabsData.data },
46-
version: TAB_DATA_VERSION,
47-
}
48-
}

src/components/common/navigation/NavigationRoutes.tsx

Lines changed: 5 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ import editorWorker from 'monaco-editor/esm/vs/editor/editor.worker?worker'
7878
import YamlWorker from '../../../yaml.worker.js?worker'
7979
import { TAB_DATA_LOCAL_STORAGE_KEY } from '../DynamicTabs/constants'
8080
import { ENVIRONMENT_DATA_FALLBACK, INITIAL_ENV_DATA_STATE } from './constants'
81-
import { ParsedTabsData, ParsedTabsDataV1 } from '../DynamicTabs/types'
81+
import { ParsedTabsData } from '../DynamicTabs/types'
8282
import { SwitchThemeDialog } from '@Pages/Shared'
8383
import { SwitchThemeDialogProps } from '@Pages/Shared/SwitchThemeDialog/types'
8484
import { EnvironmentDataStateType } from './types'
@@ -401,19 +401,10 @@ export default function NavigationRoutes() {
401401
const persistedTabs = localStorage.getItem(TAB_DATA_LOCAL_STORAGE_KEY)
402402
if (persistedTabs) {
403403
try {
404-
const parsedTabsData: ParsedTabsData | ParsedTabsDataV1 = JSON.parse(persistedTabs)
405-
if (parsedTabsData.version === 'v1') {
406-
if (
407-
location.pathname === parsedTabsData.key ||
408-
!location.pathname.startsWith(`${parsedTabsData.key}/`)
409-
) {
410-
localStorage.removeItem(TAB_DATA_LOCAL_STORAGE_KEY)
411-
}
412-
} else {
413-
const keys = Object.keys(parsedTabsData.data)
414-
if (keys.every((key) => location.pathname !== key && !location.pathname.startsWith(`${key}/`))) {
415-
localStorage.removeItem(TAB_DATA_LOCAL_STORAGE_KEY)
416-
}
404+
const parsedTabsData: ParsedTabsData = JSON.parse(persistedTabs)
405+
const keys = Object.keys(parsedTabsData.data)
406+
if (keys.every((key) => location.pathname !== key && !location.pathname.startsWith(`${key}/`))) {
407+
localStorage.removeItem(TAB_DATA_LOCAL_STORAGE_KEY)
417408
}
418409
} catch {
419410
localStorage.removeItem(TAB_DATA_LOCAL_STORAGE_KEY)

yarn.lock

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1126,10 +1126,10 @@
11261126
dependencies:
11271127
"@jridgewell/trace-mapping" "0.3.9"
11281128

1129-
"@devtron-labs/[email protected]beta-7":
1130-
version "1.11.0-beta-7"
1131-
resolved "https://registry.yarnpkg.com/@devtron-labs/devtron-fe-common-lib/-/devtron-fe-common-lib-1.11.0-beta-7.tgz#19e398fb2ceb93544954a5c54fff5d4a5c245103"
1132-
integrity sha512-ncy2smv0gsLfMpmW0l6GUezb7J1IlRQ87ZqJ1TAWu5vPc0LevIKMh5zbDBWVGlmWU75T8OCpEi1DMxU1UcN4Vg==
1129+
"@devtron-labs/[email protected]pre-6":
1130+
version "1.11.0-pre-6"
1131+
resolved "https://registry.yarnpkg.com/@devtron-labs/devtron-fe-common-lib/-/devtron-fe-common-lib-1.11.0-pre-6.tgz#e74d94c895e0fdbbe4101341d8d00444a4bdf792"
1132+
integrity sha512-nNQOtzsXbs+c0H/EcR00ucsl7X/VjeOZ3spWDcZ3PQbSC+kpi4bHkcQFyAv3H2YGf8C6QVNeoTPtmfr+r/Vsmg==
11331133
dependencies:
11341134
"@codemirror/lang-json" "6.0.1"
11351135
"@codemirror/lang-yaml" "6.1.2"

0 commit comments

Comments
 (0)