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

[Synthetics UI] [Overview] Actions #135152

Closed
2 tasks done
dominiqueclarke opened this issue Jun 24, 2022 · 4 comments · Fixed by #136992
Closed
2 tasks done

[Synthetics UI] [Overview] Actions #135152

dominiqueclarke opened this issue Jun 24, 2022 · 4 comments · Fixed by #136992
Assignees
Labels
Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability

Comments

@dominiqueclarke
Copy link
Contributor

dominiqueclarke commented Jun 24, 2022

Design

Screen Shot 2022-06-24 at 2 16 04 PM

Dependencies:

Each grid item in the overview page will display an actions menu when hovered. The menu will contain the following actions:

  • Go to monitor (Navigate to monitor details page)
  • Quick inspect (Open the monitor details flyout [Synthetics UI] [Overview] Monitor details flyout #135150)
  • Run test manually (Execute test now)
  • Edit monitor (Navigate to edit monitor page)
  • Disable monitor (Disable the monitor via the edit api)
@dominiqueclarke dominiqueclarke added the Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability label Jun 24, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@dominiqueclarke
Copy link
Contributor Author

Actions generally look good, but there is a bug that was introduced by this that needs review: #140553

Also, the edit button still goes to the Uptime edit page because this PR was merged before edit was merged. Not sure if you want to update the link before 8.5.0 goes live or if it's not that big of a deal @paulb-elastic

@paulb-elastic
Copy link
Contributor

Thanks for highlighting this @dominiqueclarke

I wouldn't worry about anything specific for 8.5 for this, however is there anything else we need to do with the other merge to ensure this does go to the right place in main/8.6? (I've tried editing in main and it still goes to Uptime, but not sure which PR you are referring to, do we need to capture that the edit should go to Synthetics?)

@dominiqueclarke
Copy link
Contributor Author

Handled in #142394. Post FF testing of this issue LGTM

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

Successfully merging a pull request may close this issue.

4 participants