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] Update notices and banners from experimental to beta #91421

Closed
paulb-elastic opened this issue Feb 15, 2021 · 10 comments · Fixed by #96886
Closed

[Uptime] Update notices and banners from experimental to beta #91421

paulb-elastic opened this issue Feb 15, 2021 · 10 comments · Fixed by #96886
Assignees
Labels
Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability test-plan-ok issue has passed test plan v7.13.0

Comments

@paulb-elastic
Copy link
Contributor

paulb-elastic commented Feb 15, 2021

During the change of phase from Experimental to Beta, we need to update some of the working and signposting that currently exists.

At the moment, there is a callout to Synthetics when you go into Uptime (although it does not specifically mention experimental, or beta). This includes a Read announcement link to https://www.elastic.co/what-is/synthetic-monitoring
image
This can be dismissed so it does not keep appearing.

There is also a callout, to Experimental... when expanding a test result on the History page:
image

This does not have a dismiss option, or a link to find out more details.

During a recent discussion, it was suggested this might be better to be shown against the monitor (further up the page, rather than against a specific test results), as well as to include a link out to find out more details.

This issue needs some input from @elastic/observability-design and @drewpost to define:

  • Does the wording on the main Uptime page need to change?
  • Should the or Read announcement link change to a different URL?
  • Where should the Experimental feature blue banner move to, and what should the wording be? Should this include the same link as the main Uptime page to find out more?
  • Which of these should be dismissable?
@paulb-elastic paulb-elastic added the Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability label Feb 15, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@paulb-elastic
Copy link
Contributor Author

cc @elastic/observability-design / @drewpost

@katrin-freihofner
Copy link
Contributor

@paulb-elastic
Summary of my discussion with @drewpost:

  • Drew will help us to get the text in the Elastic Synthetics callout updated
  • We are going to remove the small callout from each of the rows
  • We are going to add a single line underneath the headline (monitor name) with a Beta disclaimer (text comes from @drewpost ) and a link to more information (@drewpost could you please add the link here too?).

Screenshot 2021-03-03 at 11 27 02

@paulb-elastic
Copy link
Contributor Author

@drewpost we are getting closer to needing to make this change for beta - can you provide the copy and link please?

@drewpost
Copy link

drewpost commented Apr 6, 2021

Text for the warning on Monitor Details page. I want to keep it short and sweet.

Beta See More

(link to https://www.elastic.co/what-is/synthetic-monitoring )

@paulb-elastic
Copy link
Contributor Author

Thanks @drewpost
Are we keeping the Elastic Synthetics callout (in the first screenshot above) and the Read announcement link the same as it currently is?

@dominiqueclarke dominiqueclarke self-assigned this Apr 12, 2021
@dominiqueclarke
Copy link
Contributor

@drewpost @paulb-elastic What was the determination made for the Elastic Synthetics callout? I know we discussed it in a past meeting, but it doesn't appear as if the decision was documented.

@paulb-elastic
Copy link
Contributor Author

Confirmed with @drewpost in refinement that we will keep the main Synthetics call out implementation.

@drewpost
Copy link

At least until we get to GA. It's also experiment/beta/ga agnostic text so it won't require any further updating. Additionally, if a user has surpassed the message before, they will not be shown the message again after updating.

@justinkambic
Copy link
Contributor

  • No beta badge in ICMP:
    image
  • No beta badge in TCP:
    image
  • No beta badge in HTTP:
    image
  • Beta badge for Synthetic:
    image
    • the link points to the correct URL and opens in a new tab

@justinkambic justinkambic self-assigned this Apr 29, 2021
@justinkambic justinkambic added test-plan-ok issue has passed test plan and removed test-plan labels Apr 29, 2021
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 test-plan-ok issue has passed test plan v7.13.0
Projects
None yet
6 participants