New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CloudWatch: Fix query editor does not render in Explore #20909
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! Tested and it works fine.
@davkal added this to 6.5.2 milestone |
import { Input, Segment, SegmentAsync, ValidationEvents, EventsWithValidation, Switch } from '@grafana/ui'; | ||
import { CloudWatchQuery } from '../types'; | ||
import CloudWatchDatasource from '../datasource'; | ||
import { SelectableStrings } from '../types'; | ||
import { Stats, Dimensions, QueryInlineField, QueryField, Alias } from './'; | ||
|
||
export type Props = QueryEditorProps<CloudWatchDatasource, CloudWatchQuery>; | ||
export type Props = ExploreQueryFieldProps<CloudWatchDatasource, CloudWatchQuery>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok for a quick fix, but really wish we could unify these interfaces, I see very recently that ExploreQueryFieldProps has now a NEW prop in master (compared to 6.5) (absoluteRange?: ) the plan was to get rid of the differences not expand them. Can't find the PR where this was added, do you know?
history could be handled easily internally by each editor, and onBlur could be added to main interface. not sure what this absoluteRange is used for. PanelData has a timeRange property already.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
or query history could potentially be made into a more universal feature that we can shared between dashboard / explore.
* Defined explore query editor for cloudwatch * Fix types (cherry picked from commit ab3df98)
* Defined explore query editor for cloudwatch * Fix types (cherry picked from commit ab3df98)
Fixes: #20805