Skip to content

[8.18] Replace old Security API doc links with API ref links in doc link service (#223388)#224208

Merged
natasha-moore-elastic merged 1 commit intoelastic:8.18from
natasha-moore-elastic:backport/8.18/pr-223388
Jun 17, 2025
Merged

[8.18] Replace old Security API doc links with API ref links in doc link service (#223388)#224208
natasha-moore-elastic merged 1 commit intoelastic:8.18from
natasha-moore-elastic:backport/8.18/pr-223388

Conversation

@natasha-moore-elastic
Copy link
Contributor

@natasha-moore-elastic natasha-moore-elastic commented Jun 17, 2025

This will backport the following commits from 8.19 to 8.18:

…vice (elastic#223388)

This PR replaces the old Security API doc links with API refence links
in the Kibana doc link service.

[This security-docs
PR](elastic/security-docs#6872), which removes
outdated Security Detections asciidoc API docs, is failing with the
following error:

> INFO:build_docs:Bad cross-document links:
> INFO:build_docs: Kibana [8.x]:
src/platform/packages/shared/kbn-doc-links/src/get_doc_links.ts contains
broken links to:
> INFO:build_docs:   - en/security/8.x/rule-api-overview.html
> INFO:build_docs:   - en/security/8.x/signals-migration-api.htm

Replacing these links in the Kibana doc link service should unblock the
docs PR.

The links were replaced in `main` in [this
PR](elastic#219005).

---------

Co-authored-by: Lisa Cawley <lcawley@elastic.co>
Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Jacek Kolezynski <jacek.kolezynski@elastic.co>
(cherry picked from commit 3579e5c)
Copy link
Contributor

@lcawl lcawl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@natasha-moore-elastic natasha-moore-elastic merged commit 36fed06 into elastic:8.18 Jun 17, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants