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] Tooltip on "Run test" button if on-prem locations (add/edit monitor page) #137886

Closed
dominiqueclarke opened this issue Aug 2, 2022 · 6 comments · Fixed by #138863
Closed
Labels
bug Fixes for quality problems that affect the customer experience Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v8.4.0 v8.5.0

Comments

@dominiqueclarke
Copy link
Contributor

Currently, the run once button in add/edit monitor is disabled when users only select private locations. Ideally, we should hide the button entirely until we support run once. It's confusing to have it disabled always without the user being able to take any action to enable it.

@dominiqueclarke dominiqueclarke added bug Fixes for quality problems that affect the customer experience Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v8.4.0 labels Aug 2, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@dominiqueclarke dominiqueclarke changed the title [Synthetics] Remove run once button entirely on-prem until supported [Uptime] Remove run once button entirely on-prem until supported Aug 2, 2022
@paulb-elastic
Copy link
Contributor

Can't we just have a callout like we do on the main Uptime Overview page?
image

@dominiqueclarke
Copy link
Contributor Author

dominiqueclarke commented Aug 2, 2022

Can't we just have a callout like we do on the main Uptime Overview page? image

Sure thing, that works for me. Just to clarify, I was referring to the add/edit page, not the monitor overview.

@paulb-elastic
Copy link
Contributor

@dominiqueclarke for reference, I've suggested a tweak to the wording in #137898

@paulb-elastic paulb-elastic changed the title [Uptime] Remove run once button entirely on-prem until supported [Uptime] Tooltip on "Run test" button if on-prem locations (add/edit monitor page) Aug 8, 2022
@awahab07 awahab07 self-assigned this Aug 10, 2022
@shahzad31
Copy link
Contributor

For me this still remain enabled , so reopening the issue
image

@shahzad31 shahzad31 reopened this Sep 22, 2022
@awahab07
Copy link
Contributor

I tested it on Kibana main branch and found it working. Most likely the change didn't get into 8.5 Snapshot by the time it was Post FF tested.

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 v8.4.0 v8.5.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants