[Metrics UI] use global kibana time for metrics explorer in Default V…

…iew (#92520) (#92896) * use global kibana time for explorer, stop using local storage for time * fix inventory time
elastic · Feb 25, 2021 · 0fb9c81 · 0fb9c81
1 parent 953b9ad
commit 0fb9c81
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 8 deletions.
diff --git a/x-pack/plugins/infra/public/pages/metrics/inventory_view/hooks/use_waffle_view_state.ts b/x-pack/plugins/infra/public/pages/metrics/inventory_view/hooks/use_waffle_view_state.ts
@@ -78,7 +78,9 @@ export const useWaffleViewState = () => {
         region: newState.region,
         legend: newState.legend,
       });
-      if (newState.time) {
+      // if this is the "Default View" view, don't update the time range to the view's time range,
+      // this way it will use the global Kibana time or the default time already set
+      if (newState.time && newState.id !== '0') {
         setWaffleTimeState({
           currentTime: newState.time,
           isAutoReloading: newState.autoReload,
@@ -100,4 +102,5 @@ export type WaffleViewState = WaffleOptionsState & {
   time: number;
   autoReload: boolean;
   filterQuery: WaffleFiltersState;
+  id?: string;
 };
diff --git a/...ck/plugins/infra/public/pages/metrics/metrics_explorer/hooks/use_metric_explorer_state.ts b/...ck/plugins/infra/public/pages/metrics/metrics_explorer/hooks/use_metric_explorer_state.ts
@@ -24,6 +24,7 @@ export interface MetricExplorerViewState {
   chartOptions: MetricsExplorerChartOptions;
   currentTimerange: MetricsExplorerTimeOptions;
   options: MetricsExplorerOptions;
+  id?: string;
 }
 
 export const useMetricsExplorerState = (
@@ -42,6 +43,7 @@ export const useMetricsExplorerState = (
     setTimeRange,
     setOptions,
   } = useContext(MetricsExplorerOptionsContainer.Context);
+
   const { loading, error, data, loadData } = useMetricsExplorerData(
     options,
     source,
@@ -121,7 +123,11 @@ export const useMetricsExplorerState = (
         setChartOptions(vs.chartOptions);
       }
       if (vs.currentTimerange) {
-        setTimeRange(vs.currentTimerange);
+        // if this is the "Default View" view, don't update the time range to the view's time range,
+        // this way it will use the global Kibana time or the default time already set
+        if (vs.id !== '0') {
+          setTimeRange(vs.currentTimerange);
+        }
       }
       if (vs.options) {
         setOptions(vs.options);

diff --git a/...s/infra/public/pages/metrics/metrics_explorer/hooks/use_metrics_explorer_options.test.tsx b/...s/infra/public/pages/metrics/metrics_explorer/hooks/use_metrics_explorer_options.test.tsx
@@ -68,7 +68,6 @@ describe('useMetricExplorerOptions', () => {
     expect(result.current.currentTimerange).toEqual(DEFAULT_TIMERANGE);
     expect(result.current.isAutoReloading).toEqual(false);
     expect(STORE.MetricsExplorerOptions).toEqual(JSON.stringify(DEFAULT_OPTIONS));
-    expect(STORE.MetricsExplorerTimeRange).toEqual(JSON.stringify(DEFAULT_TIMERANGE));
   });
 
   it('should change the store when options update', () => {
@@ -96,7 +95,6 @@ describe('useMetricExplorerOptions', () => {
     });
     rerender();
     expect(result.current.currentTimerange).toEqual(newTimeRange);
-    expect(STORE.MetricsExplorerTimeRange).toEqual(JSON.stringify(newTimeRange));
   });
 
   it('should load from store when available', () => {

diff --git a/...plugins/infra/public/pages/metrics/metrics_explorer/hooks/use_metrics_explorer_options.ts b/...plugins/infra/public/pages/metrics/metrics_explorer/hooks/use_metrics_explorer_options.ts
@@ -163,10 +163,7 @@ export const useMetricsExplorerOptions = () => {
     'MetricsExplorerOptions',
     DEFAULT_OPTIONS
   );
-  const [currentTimerange, setTimeRange] = useStateWithLocalStorage<MetricsExplorerTimeOptions>(
-    'MetricsExplorerTimeRange',
-    defaultTimeRange
-  );
+  const [currentTimerange, setTimeRange] = useState<MetricsExplorerTimeOptions>(defaultTimeRange);
 
   useSyncKibanaTimeFilterTime(TIME_DEFAULTS, {
     from: currentTimerange.from,