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

[Fleet] Override branch in doc links #119514

Merged
merged 5 commits into from
Nov 24, 2021
Merged

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Nov 23, 2021

Summary

The documentation links in the language client integrations get a 404 (e.g. when you click any of the clients here):

image

This is occurring because those links do not use the Kibana doc link service (per
#113666 (comment) and #95389) and therefore don't have our current work-around for the lack of "main" branch documentation.

This PR therefore just overrides the branch in https://github.com/elastic/kibana/blob/main/src/plugins/custom_integrations/server/language_clients/index.ts until we replace the "master" documentation with "main".

@lcawl lcawl added release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting Team:Fleet Team label for Observability Data Collection Fleet team v8.1.0 Feature:Unified Integrations Unified Integrations view feature labels Nov 23, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

jen-huang
jen-huang previously approved these changes Nov 23, 2021
@jen-huang jen-huang dismissed their stale review November 23, 2021 20:09

Missed logic

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@lcawl lcawl merged commit 90bebce into elastic:main Nov 24, 2021
@lcawl lcawl deleted the integrations-doc-links branch November 24, 2021 02:38
@jen-huang jen-huang added auto-backport Deprecated - use backport:version if exact versions are needed and removed backport:skip This commit does not require backporting labels Nov 24, 2021
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
8.0

This backport PR will be merged automatically after passing CI.

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 24, 2021
Co-authored-by: Jen Huang <its.jenetic@gmail.com>
kibanamachine added a commit that referenced this pull request Nov 24, 2021
Co-authored-by: Jen Huang <its.jenetic@gmail.com>

Co-authored-by: Lisa Cawley <lcawley@elastic.co>
Co-authored-by: Jen Huang <its.jenetic@gmail.com>
dmlemeshko pushed a commit that referenced this pull request Nov 29, 2021
Co-authored-by: Jen Huang <its.jenetic@gmail.com>
TinLe pushed a commit to TinLe/kibana that referenced this pull request Dec 22, 2021
Co-authored-by: Jen Huang <its.jenetic@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed Feature:Unified Integrations Unified Integrations view feature release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v8.0.0 v8.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants