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.

Sign up for GitHub

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

Jump to bottom

[Infra] Change legacy metric callout "learn more" link address #190764

Closed
crespocarlos opened this issue Aug 20, 2024 · 1 comment · Fixed by #190774
Closed

[Infra] Change legacy metric callout "learn more" link address #190764

crespocarlos opened this issue Aug 20, 2024 · 1 comment · Fixed by #190774
Assignees
@crespocarlos
Labels
enhancement New value added to drive a business result Feature:ObsHosts Hosts feature within Observability good first issue low hanging fruit Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team

Comments

@crespocarlos
Copy link
Contributor

crespocarlos commented Aug 20, 2024 •
edited
Loading

Summary

The LegacyAlertMetricCallout component's "Learn more" link redirects to https://ela.st/docs-infra-host-metrics, we need to change it to point to https://ela.st/host-metrics-legacy instead

AC
@crespocarlos crespocarlos added enhancement New value added to drive a business result Feature:ObsHosts Hosts feature within Observability Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team labels Aug 20, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@crespocarlos crespocarlos added the good first issue low hanging fruit label Aug 20, 2024
@crespocarlos crespocarlos self-assigned this Aug 20, 2024
@crespocarlos crespocarlos mentioned this issue Aug 20, 2024
Merged
crespocarlos added a commit that referenced this issue Aug 21, 2024
@crespocarlos
Update learn more link address (#190774)
c11e760 
closes [#190764](#190764)
 
## Summary

Update the Learn More link on the legacy metric callout to point to
https://ela.st/host-metrics-legacy

<img width="1727" alt="image"
src="https://github.com/user-attachments/assets/67caf5d4-fcb7-4ddc-b650-b65b7bd87b49">

NODE: https://ela.st/host-metrics-legacy shortened URL redirect still
needs to be updated.

### How to test

- Start a local Kibana instance (pointing to an oblt cluster is easier)
- Navigate to Inventory > Hosts, click on a host name to navigate to the
Asset Details page
- Change the URL, adding the following queryparam: `alertMetric:rx`
- Inspect the Learn More link href
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@crespocarlos crespocarlos

Labels
enhancement New value added to drive a business result Feature:ObsHosts Hosts feature within Observability good first issue low hanging fruit Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update learn more link address crespocarlos/kibana
2 participants
@crespocarlos @elasticmachine