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

Move useTrackMetric hook to Observability Shared #157274

Closed
CoenWarmer opened this issue May 10, 2023 · 1 comment · Fixed by #157272
Closed

Move useTrackMetric hook to Observability Shared #157274

CoenWarmer opened this issue May 10, 2023 · 1 comment · Fixed by #157272
Assignees
Labels
Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge"

Comments

@CoenWarmer
Copy link
Contributor

Part of #152783

📝 Summary

Currently Observability apps use the useTrackMetric hook that's exposed from the Observability app. Since we now have created an Observability Shared app, it makes sense to have the apps use the hook exposed from there.

✅ Acceptance criteria

  • all imports of useTrackMetrics should be moved to Observability Shared
  • useTrackMetrics should be removed from Observability app
@CoenWarmer CoenWarmer self-assigned this May 10, 2023
@botelastic botelastic bot added the needs-team Issues missing a team label label May 10, 2023
@CoenWarmer CoenWarmer added the Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" label May 10, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/actionable-observability (Team: Actionable Observability)

@botelastic botelastic bot removed the needs-team Issues missing a team label label May 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge"
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants