Add /charts/series ext api #279
Conversation
|
| series: z.array(externalQueryChartSeriesSchema).refine( | ||
| series => { | ||
| const groupByFields = series[0].groupBy; | ||
| return series.every(s => _.isEqual(s.groupBy, groupByFields)); | ||
| }, | ||
| { | ||
| message: 'All series must have the same groupBy fields', | ||
| }, | ||
| ), |
| const internalSeries = series.map(s => | ||
| translateExternalSeriesToInternalSeries({ | ||
| type: 'time', // just to reuse the same fn | ||
| ...s, | ||
| }), | ||
| ); |
There was a problem hiding this comment.
nit: can do this after team validation
|
|
||
| return { | ||
| type, | ||
| data_source: table === 'metrics' ? 'metrics' : 'events', |
There was a problem hiding this comment.
do we want to make this guy camelcase ?
There was a problem hiding this comment.
🤦 yeah good point... there's a few places where I introduced the snake case. Let me do it in a follow up PR after this and the TF one is merged so I can change it everywhere at once.
No description provided.