Skip to content

Commit

Permalink
Adding backwards compatibility for older time comparison (#3676)
Browse files Browse the repository at this point in the history
  • Loading branch information
AdityaHegde authored and mindspank committed Dec 18, 2023
1 parent 6deb436 commit df87909
Showing 1 changed file with 28 additions and 1 deletion.
29 changes: 28 additions & 1 deletion web-common/src/features/dashboards/proto-state/fromProto.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,10 @@
import type { Timestamp } from "@bufbuild/protobuf";
import { LeaderboardContextColumn } from "@rilldata/web-common/features/dashboards/leaderboard-context-column";
import type { MetricsExplorerEntity } from "@rilldata/web-common/features/dashboards/stores/metrics-explorer-entity";
import { TimeRangePreset } from "@rilldata/web-common/lib/time/types";
import {
TimeComparisonOption,
TimeRangePreset,
} from "@rilldata/web-common/lib/time/types";
import type {
DashboardTimeControls,
ScrubRange,
Expand Down Expand Up @@ -66,6 +69,8 @@ export function getDashboardStateFromProto(
entity.selectedComparisonTimeRange = fromTimeRangeProto(
dashboard.compareTimeRange
);
// backwards compatibility
correctComparisonTimeRange(entity.selectedComparisonTimeRange);
}
entity.showTimeComparison = Boolean(dashboard.showTimeComparison);

Expand Down Expand Up @@ -190,6 +195,28 @@ function fromTimeRangeProto(timeRange: DashboardTimeRange) {
return selectedTimeRange;
}

function correctComparisonTimeRange(
comparisonTimeRange: DashboardTimeControls
) {
switch (comparisonTimeRange.name as string) {
case "P1D":
comparisonTimeRange.name = TimeComparisonOption.DAY;
break;
case "P1W":
comparisonTimeRange.name = TimeComparisonOption.WEEK;
break;
case "P1M":
comparisonTimeRange.name = TimeComparisonOption.MONTH;
break;
case "P3M":
comparisonTimeRange.name = TimeComparisonOption.QUARTER;
break;
case "P1Y":
comparisonTimeRange.name = TimeComparisonOption.YEAR;
break;
}
}

function fromTimeProto(timestamp: Timestamp) {
return new Date(Number(timestamp.seconds));
}
Expand Down

0 comments on commit df87909

Please sign in to comment.