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

Move config setting for By Value Embeddables to Dashboard #73870

Merged

Conversation

ThomThomson
Copy link
Contributor

@ThomThomson ThomThomson commented Jul 30, 2020

Summary

This PR moves the configuration item which determines whether or not to show the new flow for lens / visualize embeddable creation onto the dashboard start contract.

This way, all by value PRs, including #70272, #72256, and the upcoming PRs for the add to library, and unlink from library actions can refer to the same configuration setting.

Note

This config setting is temporary, and used to hide all 'by value' changes. Once all blockers to the 'by value' system are resolved (#61663, #67931, #71499), this config setting can be removed.

Checklist

Delete any items that are not applicable to this PR.

- [ ] Unit or functional tests were updated or added to match the most common scenarios

For maintainers

@ThomThomson ThomThomson added Feature:Dashboard Dashboard related features Team:Visualizations Visualization editors, elastic-charts and infrastructure v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.10.0 labels Jul 30, 2020
ThomThomson added a commit to ThomThomson/kibana that referenced this pull request Jul 30, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

page load bundle size

id value diff baseline
dashboard 678.1KB +64.0B 678.1KB

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

ThomThomson added a commit to ThomThomson/kibana that referenced this pull request Jul 30, 2020
@ThomThomson ThomThomson marked this pull request as ready for review July 31, 2020 13:59
@ThomThomson ThomThomson requested a review from a team July 31, 2020 13:59
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

Copy link
Contributor

@flash1293 flash1293 left a comment

Choose a reason for hiding this comment

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

Tested and yml setting is propagated as expected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Dashboard Dashboard related features release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Visualization editors, elastic-charts and infrastructure v7.10.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants