-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Hystrix Dashboard Eureka integration improvements #1336
Hystrix Dashboard Eureka integration improvements #1336
Conversation
- Refactoring code - Grouping instances by services - Added a new group for the first instance of each server - Updating url on chaning stream type - Update title for with application name
These changes look good to me. Anybody else that uses hystrix-dashboard willing to try this out and give @kennedyoliveira feedback? /cc mukteshkrmishra |
@mattrjacobs @kennedyoliveira : Changes looks good to me ..just tested it. |
I'd like to take a look |
Thanks @mukteshkrmishra / @spencergibb for volunteering! |
@spencergibb any feedback? |
@mattrjacobs can you merge it? |
Sure thing. Missed @spencergibb's thumbs-up - those don't trigger my email alerts :( Thanks @kennedyoliveira for the contribution, and thanks @spencergibb and @mukteshkrmishra for the review! |
@mattrjacobs Your welcome! I missed the thumbs-up too haha. |
@kennedyoliveira 👍 Here you go :) |
Hello!
Did some improvements on Hystrix Dashboard Eureka Integration, hope it helps!
The following was improved:
I guess these changes improves the overall UX, hope it helps!
New app listing:
Remove stream: