-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Visualize2Lens] Transfers the custom timerange to the converted panel #155113
Conversation
Pinging @elastic/kibana-visualizations @elastic/kibana-visualizations-external (Team:Visualizations) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Checked locally 👍
Found that also tags
are not carried over when converting to Lens, so added a new comment in the referenced issue. Not blocking this PR.
src/plugins/visualizations/public/visualize_app/utils/get_top_nav_config.tsx
Outdated
Show resolved
Hide resolved
…nav_config.tsx Co-authored-by: Marco Liberati <[email protected]>
💚 Build Succeeded
Metrics [docs]Async chunks
Page load bundle
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: |
elastic#155113) ## Summary Part of elastic#147646 It passes the custom timerange to the converted Lens panel for both by ref and by value legacy visualizations. It works for all paths: - Edit visualization--> Edit in Lens--> Replace in dashboard - Convert to Lens --> Replace in dashboard ![2](https://user-images.githubusercontent.com/17003240/233287641-82fe190d-5b92-4368-ace8-0b576a46d32a.gif) ### Checklist - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios --------- Co-authored-by: Marco Liberati <[email protected]> (cherry picked from commit 61c82dc)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…d panel (#155113) (#155979) # Backport This will backport the following commits from `main` to `8.8`: - [[Visualize2Lens] Transfers the custom timerange to the converted panel (#155113)](#155113) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Stratoula Kalafateli","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-04-27T09:11:35Z","message":"[Visualize2Lens] Transfers the custom timerange to the converted panel (#155113)\n\n## Summary\r\n\r\nPart of https://github.com/elastic/kibana/issues/147646\r\n\r\nIt passes the custom timerange to the converted Lens panel for both by\r\nref and by value legacy visualizations.\r\nIt works for all paths:\r\n- Edit visualization--> Edit in Lens--> Replace in dashboard\r\n- Convert to Lens --> Replace in dashboard\r\n\r\n\r\n![2](https://user-images.githubusercontent.com/17003240/233287641-82fe190d-5b92-4368-ace8-0b576a46d32a.gif)\r\n\r\n### Checklist\r\n\r\n- [x] [Unit or functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere updated or added to match the most common scenarios\r\n\r\n---------\r\n\r\nCo-authored-by: Marco Liberati <[email protected]>","sha":"61c82dc8682edc33bc7a8483befed136052d95dc","branchLabelMapping":{"^v8.9.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:enhancement","Feature:Visualizations","Team:Visualizations","Feature:Lens","backport:prev-minor","v8.8.0","v8.9.0"],"number":155113,"url":"https://github.com/elastic/kibana/pull/155113","mergeCommit":{"message":"[Visualize2Lens] Transfers the custom timerange to the converted panel (#155113)\n\n## Summary\r\n\r\nPart of https://github.com/elastic/kibana/issues/147646\r\n\r\nIt passes the custom timerange to the converted Lens panel for both by\r\nref and by value legacy visualizations.\r\nIt works for all paths:\r\n- Edit visualization--> Edit in Lens--> Replace in dashboard\r\n- Convert to Lens --> Replace in dashboard\r\n\r\n\r\n![2](https://user-images.githubusercontent.com/17003240/233287641-82fe190d-5b92-4368-ace8-0b576a46d32a.gif)\r\n\r\n### Checklist\r\n\r\n- [x] [Unit or functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere updated or added to match the most common scenarios\r\n\r\n---------\r\n\r\nCo-authored-by: Marco Liberati <[email protected]>","sha":"61c82dc8682edc33bc7a8483befed136052d95dc"}},"sourceBranch":"main","suggestedTargetBranches":["8.8"],"targetPullRequestStates":[{"branch":"8.8","label":"v8.8.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.9.0","labelRegex":"^v8.9.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/155113","number":155113,"mergeCommit":{"message":"[Visualize2Lens] Transfers the custom timerange to the converted panel (#155113)\n\n## Summary\r\n\r\nPart of https://github.com/elastic/kibana/issues/147646\r\n\r\nIt passes the custom timerange to the converted Lens panel for both by\r\nref and by value legacy visualizations.\r\nIt works for all paths:\r\n- Edit visualization--> Edit in Lens--> Replace in dashboard\r\n- Convert to Lens --> Replace in dashboard\r\n\r\n\r\n![2](https://user-images.githubusercontent.com/17003240/233287641-82fe190d-5b92-4368-ace8-0b576a46d32a.gif)\r\n\r\n### Checklist\r\n\r\n- [x] [Unit or functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere updated or added to match the most common scenarios\r\n\r\n---------\r\n\r\nCo-authored-by: Marco Liberati <[email protected]>","sha":"61c82dc8682edc33bc7a8483befed136052d95dc"}}]}] BACKPORT--> Co-authored-by: Stratoula Kalafateli <[email protected]>
Summary
Part of #147646
It passes the custom timerange to the converted Lens panel for both by ref and by value legacy visualizations.
It works for all paths:
Checklist