Add noop default to fetchTimeSeries

Layouts don't render with an `editQueryStatus` to dispatch. The noop is
required to have auto refreshing layouts.
pull/10616/head
Delmer Reed 2018-07-11 16:38:05 -04:00
parent 9ac7c21ee9
commit a94a00cd7f
2 changed files with 9 additions and 4 deletions

View File

@ -20,7 +20,6 @@ import {
stylePixelOffset,
defaultColumnWidth,
} from 'src/data_explorer/constants/table'
import {noop} from 'src/shared/actions/app'
import {Source, Template} from 'src/types'
@ -215,7 +214,12 @@ class ChronoTable extends PureComponent<Props, State> {
return _.get(series, `${activeSeriesIndex}`, emptySeries)
}
private fetchCellData = async ({source, query, templates}: Props) => {
private fetchCellData = async ({
source,
query,
templates,
editQueryStatus,
}: Props) => {
if (!query || !query.text) {
return
}
@ -230,7 +234,7 @@ class ChronoTable extends PureComponent<Props, State> {
[query],
NULL_RESOLUTION,
templates,
noop
editQueryStatus
)
const results = getDeep(queriesResponse, '0.results', [])

View File

@ -9,6 +9,7 @@ import {analyzeQueries} from 'src/shared/apis'
import {DEFAULT_DURATION_MS} from 'src/shared/constants'
import replaceTemplates, {replaceInterval} from 'src/tempVars/utils/replace'
import {proxy} from 'src/utils/queryUrlGenerator'
import {noop} from 'src/shared/actions/app'
import {Source} from 'src/types'
@ -27,7 +28,7 @@ export const fetchTimeSeries = async (
queries: Query[],
resolution: number,
templates: Template[],
editQueryStatus: () => any
editQueryStatus: () => any = noop
) => {
const timeSeriesPromises = queries.map(async query => {
const {database, rp} = query