forked from grafana-tools/sdk
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Update to upstream, fix statPanel marshalling #13
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Add FieldConfig to GraphPanel * add Links to FieldConfig
* Added Tags to Target object (grafana-tools#169) * correct annotations * add quotes to annotations * added unit test for Target.Tags * make unit tests work - correct JSON for target.tags, fatal error on assertion fail * move Target.tags test logic to unmarshal test Co-authored-by: Tank <[email protected]>
Add TextPanel tests that test whether it actually works on Grafana.
…-tools#180) * GroupBy should be singular in the JSON * fix json declaration for groupBy Co-authored-by: Tank <[email protected]>
* feat: dashboard versions API support * chore: add test and update style * improve naming Co-authored-by: Giedrius Statkevičius <[email protected]> Co-authored-by: Vincent Miszczak <[email protected]> Co-authored-by: Giedrius Statkevičius <[email protected]>
* Added team REST * Fixed failing tests * Use new query parameter pattern and refactored GetTeamByName * Fixed failing integration tests * Dont return team as pointer for GetTeamByName * Update team.go Co-authored-by: Giedrius Statkevičius <[email protected]> * Added team not found error message * Update rest-team.go Co-authored-by: Giedrius Statkevičius <[email protected]>
* Added folder permission REST support * Removed unneeded UpdateFolderPermissionRequest * Handle return value of defer to fix linting * Removed UpdateFolderPermissionRequest
* Adding additional fields to match: https://grafana.com/docs/grafana/latest/http_api/data_source/ * Addressing code review comments.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR:
FieldConfig
field forStatPanel
grafana-tools/sdk#183) is not merged in the upstream project.this is required to fix kubermatic/kubermatic#8506