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

Add guided_cityindex as the default dashboard #3

Merged
merged 8 commits into from
Mar 20, 2014

Conversation

mrdavidlaing
Copy link

The default page when hitting kibana should provide guidance on how dashboards work in the context of the City Index data

@mrdavidlaing
Copy link
Author

@dpb587, @sopel - This is deployed to http://kibana-pr3.monitor-cloud.cityindextest5.co.uk/

Could you review and give me some feedback please

@dpb587
Copy link

dpb587 commented Mar 20, 2014

I think you should better distinguish between "queries" and "filters", or give slightly different examples. For example, you suggest to add a Levels filter, but there's a Levels reference in the query and that might cause confusion. Perhaps something along the lines of "queries let you segment your results" while "filters let you restrict all your results" with an example of "filter by environment so PPE isn't cluttering the results".

@sopel
Copy link

sopel commented Mar 20, 2014

With the changes in place, I think this is a pretty nice approach to introduce users to the system, at least assuming they start with the top level on Kibana page (if you've never seen it before, the Dasboard Index might be a bit visually confusing still, as clear as it is for those in the know) - great job :)

@mrdavidlaing
Copy link
Author

My Gradle experiment started well; the gradle-grunt-plugin encapsulates installing node, npm, grunt and running grunt build really elegantly.

I had high hopes for the cf-gradle-plugin; which then crashed and burnt because it can only upload WAR files. Sigh.

Oh well. Time to roll back.

mrdavidlaing added a commit that referenced this pull request Mar 20, 2014
Add guided_cityindex as the default dashboard
@mrdavidlaing mrdavidlaing merged commit a0a1671 into master Mar 20, 2014
@mrdavidlaing mrdavidlaing deleted the guided_cityindex_dashboard branch March 20, 2014 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants