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

[Docs] fixing KibanaPageTemplate docs #100104

Merged
merged 2 commits into from
May 14, 2021
Merged

Conversation

myasonik
Copy link
Contributor

Updating all instances of KibanaPageLayout to KibanaPageTemplate and fixing any related slugs/ids

@myasonik myasonik added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes docs v7.14.0 auto-backport Deprecated - use backport:version if exact versions are needed labels May 13, 2021
@myasonik myasonik requested a review from gtback May 13, 2021 23:48
Copy link
Contributor

@glitteringkatie glitteringkatie left a comment

Choose a reason for hiding this comment

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

Unsure about all that broken build stuff 👀 But from a docs standpoint, this works and I'm able to click the link in building_blocks and move over to the kpt page! AND I learned a really cool thing that Just Works™ in at least the gatsby system: by using the following object in my sources-dev.json I was able to test your branch without having kibana locally 🤩

    {
      "type": "github",
      "location": "myasonik/kibana",
      "branch": "fix-docs"
    }

@gtback
Copy link
Member

gtback commented May 14, 2021

@elasticmachine merge upstream

Copy link
Member

@gtback gtback left a comment

Choose a reason for hiding this comment

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

LGTM! I don't see this page (with either the old or the new ID) in the navigation anywhere, so we can add that in separately.

@kibanamachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Metrics [docs]

✅ unchanged

History

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

@myasonik myasonik merged commit 2ba09e4 into elastic:master May 14, 2021
@myasonik myasonik deleted the fix-docs branch May 14, 2021 17:59
kibanamachine added a commit to kibanamachine/kibana that referenced this pull request May 14, 2021
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
7.x

This backport PR will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request May 14, 2021
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: Michail Yasonik <michail.yasonik@elastic.co>
yctercero pushed a commit to yctercero/kibana that referenced this pull request May 17, 2021
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
yctercero pushed a commit to yctercero/kibana that referenced this pull request May 25, 2021
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.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 docs release_note:skip Skip the PR/issue when compiling release notes v7.14.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants