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

[Uptime] Filter bar missing Name. Values for ID/URL make little sense #42096

Closed
andrewvc opened this issue Jul 26, 2019 · 2 comments
Closed

[Uptime] Filter bar missing Name. Values for ID/URL make little sense #42096

andrewvc opened this issue Jul 26, 2019 · 2 comments
Labels
bug Fixes for quality problems that affect the customer experience Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability

Comments

@andrewvc
Copy link
Contributor

As @jamiesmith pointed out we seem to be missing monitor names in the filter bar dropdown (see SS below).

This is a bug and a regression strictly speaking, however I don't believe it ever made sense to show a list of names there. It would only correspond to the name column of the larger list below. The same goes for ID and URL. Filtering by items with a cardinality of 1 doesn't make a lot of sense.

The real utility of these items in the filter bar is that using their search boxes is easier than typing/remembering monitor.name:"my name". To that end I propose we stop showing values in the dropdown for everything but port and type which are actually enumerable and have many monitors to a single value, and just use the others for their search boxes.

I'd propose that we show the search boxes for these fields, and let users input arbitrary strings there to add to the query. This probably dovetails with the Kuery bar work @jasonrhodes is working on. I'd very much appreciate your opinion here @jasonrhodes given that.

image

@katrin-freihofner wdyt?

@andrewvc andrewvc added bug Fixes for quality problems that affect the customer experience Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability labels Jul 26, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime

@katrin-freihofner
Copy link
Contributor

I had a quick chat with @andrewvc and we decided to drop filters for name, ID and URL for now as there is a PR that will solve this anyhow: #41588

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability
Projects
None yet
Development

No branches or pull requests

3 participants