Skip to content
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

Read grafana response #33

Merged
merged 6 commits into from
Sep 22, 2019
Merged

Read grafana response #33

merged 6 commits into from
Sep 22, 2019

Conversation

thevan4
Copy link
Contributor

@thevan4 thevan4 commented Sep 20, 2019

The answer to the successful creation of the dashboard is not ignored.
Fields added to the StatusMessage structure: UID and URL

I needed these changes because my microservice should return links to automatically generated dashboards.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.09%) to 22.153% when pulling 6b2b2f6 on thevan4:read-grafana-response into 2c31604 on grafana-tools:master.

@grafov
Copy link
Member

grafov commented Sep 22, 2019

It looks ok, thank you @thevan4!

@grafov grafov merged commit 5b510d4 into grafana-tools:master Sep 22, 2019
@thevan4 thevan4 deleted the read-grafana-response branch September 22, 2019 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants