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

[Discover] Migrate from the legacy Get alert API to the new one #192558

Closed
cnasikas opened this issue Sep 11, 2024 · 2 comments · Fixed by #203148
Closed

[Discover] Migrate from the legacy Get alert API to the new one #192558

cnasikas opened this issue Sep 11, 2024 · 2 comments · Fixed by #203148
Assignees
Labels
Feature:Discover Discover Application impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. technical debt Improvement of the software architecture and operational architecture v9.0.0

Comments

@cnasikas
Copy link
Member

The Discover application uses a legacy alerting API for fetching the details of a rule. Specifically, it uses the /api/alerts/alert/<id>. The legacy endpoint will be removed in 9.0. The non-legacy Get rule API can be used instead.

@cnasikas cnasikas added the Feature:Discover Discover Application label Sep 11, 2024
@botelastic botelastic bot added the needs-team Issues missing a team label label Sep 11, 2024
@kertal kertal added the Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. label Sep 11, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-data-discovery (Team:DataDiscovery)

@botelastic botelastic bot removed the needs-team Issues missing a team label label Sep 11, 2024
@kertal
Copy link
Member

kertal commented Sep 12, 2024

Confirmed that we need to migrate it 👍

@kertal kertal added impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. blocked technical debt Improvement of the software architecture and operational architecture v9.0.0 and removed blocked triage_needed labels Sep 12, 2024
@cnasikas cnasikas changed the title [Discover] Migrate from the legacy alerting API to the new one [Discover] Migrate from the legacy Get alert API to the new one Sep 13, 2024
@kertal kertal self-assigned this Dec 12, 2024
@cnasikas cnasikas linked a pull request Dec 12, 2024 that will close this issue
1 task
JoseLuisGJ pushed a commit to JoseLuisGJ/kibana that referenced this issue Dec 19, 2024
## Summary

Resolves elastic#195179
Resolves elastic#192558

This PR deletes deprecated legacy alerts routes `api/alerts/alert` in
v9.0.
It also updates docs to reflect the same.


### Checklist

- [x]
[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)
was added for features that require explanation or tutorials



### Release notes
Deleted deprecated alerts routes.

---------

Co-authored-by: kibanamachine <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this issue Jan 13, 2025
## Summary

Resolves elastic#195179
Resolves elastic#192558

This PR deletes deprecated legacy alerts routes `api/alerts/alert` in
v9.0.
It also updates docs to reflect the same.


### Checklist

- [x]
[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)
was added for features that require explanation or tutorials



### Release notes
Deleted deprecated alerts routes.

---------

Co-authored-by: kibanamachine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Discover Discover Application impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. technical debt Improvement of the software architecture and operational architecture v9.0.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants