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 type to monitors details page for new browser type #254

Closed
andrewvc opened this issue Sep 29, 2020 · 1 comment · Fixed by elastic/kibana#79556
Closed

Add type to monitors details page for new browser type #254

andrewvc opened this issue Sep 29, 2020 · 1 comment · Fixed by elastic/kibana#79556
Labels
enhancement New feature or request test-plan test-plan-ok Indicates an issue has been tested for release v7.10.0

Comments

@andrewvc
Copy link
Contributor

On the monitor details page we don't show the monitor type (see current SS below). This hasn't been a problem so far because the URL scheme effectively is the type. With real browsers though it's ambiguous since the URL will be HTTP. CC @drewpost @katrin-freihofner . I propose we add a new type field just below URL containing icmp|tcp|http|browser.

image

@andrewvc andrewvc added the enhancement New feature or request label Sep 29, 2020
@shahzad31
Copy link

shahzad31 commented Oct 5, 2020

instead of adding a new row, i propose we use an EuiBadge to display HTTP/BROWSER or ICMP in front of url value or maybe in the title. It can also be before "Up in 3 Location" text

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request test-plan test-plan-ok Indicates an issue has been tested for release v7.10.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants