fix(chart): support all time and date types for X axis

Optimising data type rendering #24
GreptimeTeam · Dec 9, 2022 · 2d3664b · 2d3664b
1 parent 68e956f
commit 2d3664b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/store/modules/code-run/index.ts b/src/store/modules/code-run/index.ts
@@ -1,6 +1,7 @@
 import { getSqlResult } from '@/api/editor'
 import { Message } from '@arco-design/web-vue'
 import { defineStore } from 'pinia'
+import { dateTypes } from '@/views/dashboard/data-explorer/components/data-view/config'
 
 // TODO: Add all the types we decide instead of ECharts if needed in the future.
 // const TYPE_MAP: any = {
@@ -15,7 +16,7 @@ const getDimensionsAndXName = (elements: any) => {
   let xAxisName = ''
   let findTimeFlag = false
   elements.forEach((element: any) => {
-    if (!findTimeFlag && element.data_type === 'Timestamp') {
+    if (!findTimeFlag && dateTypes.find((type: string) => type === element.data_type)) {
       findTimeFlag = true
       xAxisName = element.name
     }

diff --git a/src/views/dashboard/data-explorer/components/data-view/config.ts b/src/views/dashboard/data-explorer/components/data-view/config.ts
@@ -30,3 +30,5 @@ export const numberTypes = [
   'Float32',
   'Float64',
 ]
+
+export const dateTypes = ['Date', 'DateTime', 'Timestamp']