-
Notifications
You must be signed in to change notification settings - Fork 218
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
add data_source_dashboard #359
add data_source_dashboard #359
Conversation
…a into Library-Elements
5d85cba
to
5b3200c
Compare
5b3200c
to
e25db98
Compare
@julienduchesne ready for review |
@inkel hey Leandro would you have time to review my work here? My coworkers and i are eager to get user-created dashboards into Terraform. |
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.
This is looking good! Just a few comments!
Co-authored-by: Julien Duchesne <julienduchesne@live.com>
Co-authored-by: Julien Duchesne <julienduchesne@live.com>
This reverts commit ad67e77.
This reverts commit a6611a8.
This reverts commit ec424f8.
This reverts commit 42cbc03.
This reverts commit 67cf749.
This reverts commit b8a64bf.
…ider-grafana into data_source_dashboard
…-provider-grafana into data_source_dashboard
…-provider-grafana into data_source_dashboard
@julienduchesne ready for review |
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.
Hey @justinTM sorry for the delay. Changes LGTM but as @julienduchesne pointed out, that additional test could add great value.
@inkel done! |
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.
LGTM. @julienduchesne wanna chime in?
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.
LGTM! Good work @justinTM
No description provided.