Skip to content

Conversation

@lcawl
Copy link
Contributor

@lcawl lcawl commented Aug 25, 2023

Relates to elastic/docs#2743

There are hard-coded links that seem to be using an inappropriate format and result in errors like this:

16:35:51 INFO:build_docs: /tmp/docsbuild/target_repo/html/en/security/8.10/threat-intel-windows-registry-indicator-match.html contains broken links to:
16:35:51 INFO:build_docs: - en/security/current/es-threat-intel-integrations.html#agent-ti-integration),
16:35:51 INFO:build_docs: - en/security/current/es-threat-intel-integrations.html#custom-ti-integration).
16:35:51 INFO:build_docs: - en/security/current/es-threat-intel-integrations.html#ti-mod-integration),
16:35:51 INFO:build_docs: - en/security/current/es-threat-intel-integrations.html).
16:35:51 INFO:build_docs: - en/security/master/invest-guide-run-osquery.html)
...
16:35:51 INFO:build_docs: /tmp/docsbuild/target_repo/html/en/security/8.10/suspicious-process-spawned-from-motd-detected.html contains broken links to:
16:35:51 INFO:build_docs: - en/security/current/osquery-placeholder-fields.html)
16:35:51 INFO:build_docs: - en/security/master/invest-guide-run-osquery.html)

To get the V8.10 documentation working, I am converting these all to internal links (<<id,label>>) however if these are generated files the scripts that generate them will need to be fixed too.

@github-actions
Copy link

Documentation previews:

@lcawl lcawl added the v8.10.0 label Aug 25, 2023
@lcawl lcawl marked this pull request as ready for review August 25, 2023 00:36
@lcawl lcawl merged commit 799c406 into elastic:8.10 Aug 25, 2023
@lcawl lcawl deleted the 810-broken-links branch August 25, 2023 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant