From 013057dc239efdae52c2593d4cd5649fe0739820 Mon Sep 17 00:00:00 2001 From: Marc Bachmann Date: Wed, 21 Apr 2021 21:13:22 +0200 Subject: [PATCH] Fix chart upgrade screen by coercing the chart version to semver (#2584) Signed-off-by: Marc Bachmann --- .../components/+apps-helm-charts/helm-chart.store.ts | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/renderer/components/+apps-helm-charts/helm-chart.store.ts b/src/renderer/components/+apps-helm-charts/helm-chart.store.ts index 70c130dca5..25559a9711 100644 --- a/src/renderer/components/+apps-helm-charts/helm-chart.store.ts +++ b/src/renderer/components/+apps-helm-charts/helm-chart.store.ts @@ -34,7 +34,10 @@ export class HelmChartStore extends ItemStore { protected sortVersions = (versions: IChartVersion[]) => { return versions.sort((first, second) => { - return semver.compare(second.version, first.version); + const firstVersion = semver.coerce(first.version || 0); + const secondVersion = semver.coerce(second.version || 0); + + return semver.compare(secondVersion, firstVersion); }); };