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

Restore vega.enableExternalUrls documentation #42820

Merged
merged 1 commit into from
Aug 8, 2019

Conversation

laraMorenoIgle
Copy link
Contributor

vega.enableExternalUrls setting was removed in the docs for all 7.x versions.
After having double checked with Kibana team that such setting is actually supported, I could conclude that it was missing in the documentation.
So I have added the setting explanation in the documentation.

vega.enableExternalUrls setting missing but it still supported, so needed to be added
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@gchaps gchaps self-requested a review August 8, 2019 16:06
@gchaps gchaps added release_note:skip Skip the PR/issue when compiling release notes Team:Docs labels Aug 8, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-docs

@gchaps gchaps added the v8.0.0 label Aug 8, 2019
Copy link
Contributor

@nyurik nyurik left a comment

Choose a reason for hiding this comment

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

thanks for adding it back!

Copy link
Contributor

@gchaps gchaps left a comment

Choose a reason for hiding this comment

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

Built the docs locally. LGTM.

Please backport this change to the 7.x docs.

@laraMorenoIgle laraMorenoIgle merged commit ef8072d into elastic:master Aug 8, 2019
@timroes timroes changed the title Update settings.asciidoc Restore vega.enableExternalUrls documentation Aug 9, 2019
timroes pushed a commit to timroes/kibana that referenced this pull request Aug 9, 2019
vega.enableExternalUrls setting missing but it still supported, so needed to be added
@timroes
Copy link
Contributor

timroes commented Aug 9, 2019

I'll take care of backporting, Lara.

timroes pushed a commit to timroes/kibana that referenced this pull request Aug 9, 2019
vega.enableExternalUrls setting missing but it still supported, so needed to be added
timroes pushed a commit that referenced this pull request Aug 9, 2019
vega.enableExternalUrls setting missing but it still supported, so needed to be added
timroes pushed a commit that referenced this pull request Aug 9, 2019
vega.enableExternalUrls setting missing but it still supported, so needed to be added
timroes pushed a commit to timroes/kibana that referenced this pull request Aug 9, 2019
vega.enableExternalUrls setting missing but it still supported, so needed to be added
timroes pushed a commit that referenced this pull request Aug 9, 2019
vega.enableExternalUrls setting missing but it still supported, so needed to be added
timroes pushed a commit to timroes/kibana that referenced this pull request Aug 9, 2019
vega.enableExternalUrls setting missing but it still supported, so needed to be added
timroes pushed a commit to timroes/kibana that referenced this pull request Aug 9, 2019
vega.enableExternalUrls setting missing but it still supported, so needed to be added
timroes pushed a commit that referenced this pull request Aug 9, 2019
vega.enableExternalUrls setting missing but it still supported, so needed to be added
timroes pushed a commit that referenced this pull request Aug 9, 2019
vega.enableExternalUrls setting missing but it still supported, so needed to be added
@laraMorenoIgle
Copy link
Contributor Author

Thanks @timroes !! I was trying to do it myself using the backport with no success! So, I really appreciate that you take care of it!

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:Docs v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants