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

Allow node filtering via URL Hash #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

chendo
Copy link

@chendo chendo commented Mar 5, 2013

I had weird issues when I tried to add a filter text field so I implemented it via a hash. The use case for us was we use have client nodes on our app servers and all the client node stats are just clutter.

We'd access it with index.html#master:true for example.

I'm not really a JS dev and have zero experience with Ember so this is what I managed to hack up in a few minutes.

@karmi
Copy link
Owner

karmi commented Mar 5, 2013

I'm not really a JS dev and have zero experience with Ember so this is what I managed to hack up in a few minutes.

The way it should be done, I think, is having an option to tick, “[] Show master nodes, [] Show data nodes”, etc.

The real problem is that I have to find enough time to pick up Paramedic after long time, upgrade it to latest Ember, and make the code a bit more sane :)

@chendo
Copy link
Author

chendo commented Mar 5, 2013

data:true doesn't return any servers for me (still on 0.19.10), so not sure if that's a bug or you can't query that particular attribute. master:true shows all our data nodes.

Heh it's usually like that :)

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.

2 participants