-
Notifications
You must be signed in to change notification settings - Fork 11
feat: explore query builder #981
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
Conversation
Codecov Report
@@ Coverage Diff @@
## main #981 +/- ##
==========================================
+ Coverage 85.05% 85.07% +0.01%
==========================================
Files 812 814 +2
Lines 16756 16773 +17
Branches 2169 2169
==========================================
+ Hits 14252 14269 +17
Misses 2473 2473
Partials 31 31
Continue to review full report at Codecov.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
||
private getAnyIntervalSelections(request: GraphQlExploreRequest): GraphQlSelection[] { | ||
return request.interval ? [{ path: 'intervalStart', alias: INTERVAL_START_QUERY_KEY }] : []; | ||
return this.exploreGraphQlQueryBuilderService.buildResponse(response, request); |
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.
Query is just the query. If you want to include response code too, then <>RequestResponseBuilder
would have been more clear. We can change it later :)
Description
Explore query builder
Testing
Local testing done.
Checklist: