-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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] Add callout for synthetics UI #79563
[Uptime] Add callout for synthetics UI #79563
Conversation
Pinging @elastic/uptime (Team:uptime) |
@elasticmachine merge upstream |
<p> | ||
<FormattedMessage | ||
id="xpack.uptime.overview.pageHeader.syntheticsCallout.content" | ||
defaultMessage="Elastic Uptime now supports synthetic browser monitors! Learn how to use them {here}." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@drewpost thoughts on this copy? and the URL below?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can I make a small design suggestion for the callout layout? I would like to suggest that we make the link to the synthetics elastic.co page the primary link and allow for users to dismiss the callout as a secondary option. I don't think the small inline link is going to be clicked, thereby making the dismiss option the most common action. Perhaps something like this? I'm sure that we can beef up the copy as well of this announcement 😊
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@andrewvc "Uptime is now previewing support for scripted multi-step availability checks. This means you can interact with elements of a webpage and check the availability of an entire journey (such as making a purchase or signing into a system) instead of just a simple single page up/down check. Please click below to read more and, if you'd like to be one of the first to use these capabilities, you can download our preview synthetics agent and view your synthetic checks in Uptime"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@andrewvc https://www.elastic.co/what-is/synthetic-monitoring That's the link
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM pending copy / URL approval from @drewpost
@drewpost I think you meant to say: 😉 |
@elasticmachine merge upstream |
💚 Build SucceededMetrics [docs]@kbn/optimizer bundle module count
async chunks size
History
To update your PR or re-run it, just comment with: |
* Add callout for synthetics UI. * Refresh outdated test snapshots. * Test synthetics callout component. * Update callout copy. * Update URL for announcement link. * Update test snapshots. Co-authored-by: Kibana Machine <[email protected]>
* Add callout for synthetics UI. * Refresh outdated test snapshots. * Test synthetics callout component. * Update callout copy. * Update URL for announcement link. * Update test snapshots. Co-authored-by: Kibana Machine <[email protected]> Co-authored-by: Kibana Machine <[email protected]>
Back ported to: |
Summary
Resolves elastic/uptime#255.
Adds a callout to the top of the overview page in the Uptime app. When dismissed, a
localStorage
key is updated to hide the callout permanently.Working copy is shown here:
Checklist
Delete any items that are not applicable to this PR.
For maintainers