Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

Grafana 4 cosmetic adaptations + config panel #40

Merged
merged 3 commits into from
Feb 6, 2017

Conversation

samuelbenhamou
Copy link

No description provided.

@@ -0,0 +1,16 @@
/// <reference path="/usr/share/grafana/public/app/headers/common.d.ts" />
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should definitely be a relative path. Is it really needed? Actually I made it work without imports and reference. Please remove those.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to clean this up myself in seperate branch. Thank you.

@jifwin
Copy link
Contributor

jifwin commented Jan 30, 2017

Thank you for this PR, it makes good progress in UI. However it still needs some polish. PRs welcomed :).

@jifwin jifwin changed the base branch from grafana4 to pr40_cleanup February 6, 2017 22:29
@jifwin jifwin closed this Feb 6, 2017
@jifwin jifwin reopened this Feb 6, 2017
@jifwin jifwin closed this Feb 6, 2017
@jifwin jifwin reopened this Feb 6, 2017
@jifwin jifwin merged commit 8d335d9 into grafana:pr40_cleanup Feb 6, 2017
@samuelbenhamou
Copy link
Author

sorry did not have much time to look at it. Great you did the cleanup. I'll test the new branch and check what's not working

@gdlx gdlx mentioned this pull request Oct 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants