Remove axes normalizer
With the `bounds` returning from the backend and the values saved as strings there's no longer a need to normalize cell state.pull/10616/head
parent
f7191b1cba
commit
1a1b077a90
|
@ -34,7 +34,6 @@ class CellEditorOverlay extends Component {
|
||||||
this.handleSetActiveQueryIndex = ::this.handleSetActiveQueryIndex
|
this.handleSetActiveQueryIndex = ::this.handleSetActiveQueryIndex
|
||||||
this.handleEditRawText = ::this.handleEditRawText
|
this.handleEditRawText = ::this.handleEditRawText
|
||||||
this.handleSetRange = ::this.handleSetRange
|
this.handleSetRange = ::this.handleSetRange
|
||||||
this.normalizeAxes = ::this.normalizeAxes
|
|
||||||
|
|
||||||
const {cell: {name, type, queries, axes}} = props
|
const {cell: {name, type, queries, axes}} = props
|
||||||
|
|
||||||
|
@ -82,7 +81,6 @@ class CellEditorOverlay extends Component {
|
||||||
handleSetRange(e) {
|
handleSetRange(e) {
|
||||||
const {min, max} = e.target.form
|
const {min, max} = e.target.form
|
||||||
|
|
||||||
// TODO: handle "" for min and max value
|
|
||||||
this.setState({
|
this.setState({
|
||||||
axes: {
|
axes: {
|
||||||
y: {
|
y: {
|
||||||
|
@ -111,10 +109,10 @@ class CellEditorOverlay extends Component {
|
||||||
queriesWorkingDraft,
|
queriesWorkingDraft,
|
||||||
cellWorkingType: type,
|
cellWorkingType: type,
|
||||||
cellWorkingName: name,
|
cellWorkingName: name,
|
||||||
|
axes,
|
||||||
} = this.state
|
} = this.state
|
||||||
|
|
||||||
const {cell} = this.props
|
const {cell} = this.props
|
||||||
const axes = this.normalizeAxes()
|
|
||||||
|
|
||||||
const queries = queriesWorkingDraft.map(q => {
|
const queries = queriesWorkingDraft.map(q => {
|
||||||
const timeRange = q.range || {upper: null, lower: ':dashboardTime:'}
|
const timeRange = q.range || {upper: null, lower: ':dashboardTime:'}
|
||||||
|
@ -137,22 +135,6 @@ class CellEditorOverlay extends Component {
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
||||||
normalizeAxes() {
|
|
||||||
const axes = this.state.axes
|
|
||||||
const bounds = _.get(axes, ['y', 'bounds'], false)
|
|
||||||
if (!bounds && !bounds.length) {
|
|
||||||
return {...axes, y: {bounds: []}}
|
|
||||||
}
|
|
||||||
|
|
||||||
const [min, max] = bounds
|
|
||||||
if (min === '' || max === '') {
|
|
||||||
// TODO: throw requirement error
|
|
||||||
return
|
|
||||||
}
|
|
||||||
|
|
||||||
return {...axes, y: {bounds: [+min, +max]}}
|
|
||||||
}
|
|
||||||
|
|
||||||
handleSelectGraphType(graphType) {
|
handleSelectGraphType(graphType) {
|
||||||
this.setState({cellWorkingType: graphType})
|
this.setState({cellWorkingType: graphType})
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue