kibana icon indicating copy to clipboard operation
kibana copied to clipboard

[Embeddables Rebuild] Migrate Visualize

Open Zacqary opened this issue 1 year ago • 69 comments
trafficstars

Summary

Closes #174958

This migrates the Visualize embeddable to the new React Embeddable framework.

Migrated:

  • Edit visualization action
  • Convert to lens action
  • Extracting/injecting references on serialize and deserialize
  • Inspector adapters
  • Dashboard settings
  • Drilldown support
  • Timeslice/time slider support
  • Custom time ranges

Also adds deprecation statements to legacy Embeddable factories

In a second PR, we'll move the embeddable folder to legacy/embeddable and rename react_embeddable to embeddable. I don't know if git will be able to diff that change in a comprehensible way in this PR, so I want to save it for the next one.

Checklist

Zacqary avatar May 10 '24 20:05 Zacqary

:robot: GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

ghost avatar May 10 '24 20:05 ghost

Pinging @elastic/kibana-presentation (Team:Presentation)

elasticmachine avatar May 13 '24 15:05 elasticmachine

@elasticmachine merge upstream

Zacqary avatar May 13 '24 15:05 Zacqary

@elasticmachine merge upstream

Zacqary avatar May 15 '24 16:05 Zacqary

/ci

Zacqary avatar May 15 '24 16:05 Zacqary

/ci

Zacqary avatar May 22 '24 17:05 Zacqary

/ci

Zacqary avatar May 22 '24 18:05 Zacqary

/ci

Zacqary avatar May 22 '24 19:05 Zacqary

/ci

Zacqary avatar May 23 '24 16:05 Zacqary

/ci

Zacqary avatar May 23 '24 20:05 Zacqary

/ci

Zacqary avatar May 28 '24 16:05 Zacqary

/ci

Zacqary avatar May 28 '24 16:05 Zacqary

/ci

Zacqary avatar May 29 '24 21:05 Zacqary

/ci

Zacqary avatar May 30 '24 18:05 Zacqary

/ci

Zacqary avatar May 30 '24 20:05 Zacqary

/ci

Zacqary avatar May 31 '24 17:05 Zacqary

/ci

Zacqary avatar May 31 '24 17:05 Zacqary

/ci

Zacqary avatar May 31 '24 23:05 Zacqary

Making notes here of what still remains to be done:

  • [ ] uiState needs to be properly passed into the DefaultEditor component
  • [ ] ~Area aggregations are throwing errors in the xyChart component, need to figure out why this is~ This has stopped happening??
  • [ ] PDF rendering still breaks a lot of CI tests
  • [ ] Visualizations added using Add To Library need to open the Saved Object editor (visualize_editor.tsx), not the By Value editor
  • [ ] By Value editor works for loading visualizations, but index pattern data isn't getting saved properly causing them to error out on the dashboard

Zacqary avatar Jun 07 '24 15:06 Zacqary

Status report on tile types:

Fully migrated

  • Text markdown
  • TSVB
  • Tag cloud
  • Line
  • Data table
  • Vertical bar
  • Horizontal bar
  • Pie
  • Timelion

Failing on UI State error

  • Gauge
  • Goal
  • Metric
  • Heat map

Zacqary avatar Jun 07 '24 15:06 Zacqary

/ci

Zacqary avatar Jul 02 '24 21:07 Zacqary

/ci

Zacqary avatar Jul 02 '24 21:07 Zacqary

/ci

Zacqary avatar Jul 08 '24 21:07 Zacqary

/ci

Zacqary avatar Jul 08 '24 21:07 Zacqary

Continuing to fix bugs, some existing, some from changes introduced from main. Known issues at the moment:

  • [x] TSVB visualizations fail to save to library
  • [x] Saving a vis from visualize and adding it to an existing dashboard doesn't transfer it to the dashboard correctly
  • [x] Creating a TSVB panel from the dashboard doesn't transfer it to the dashboard
  • [x] Saved searches don't save
  • [x] Cannot break link to saved search from visualize editor sidebar

Zacqary avatar Jul 12 '24 17:07 Zacqary

Known issues from latest merge:

  • [x] Add from library doesn't load the vis title
  • [x] Save to library breaks saved search
  • [x] Save TSVB to library and add to dashboard fails to redirect and add to dashboard

Zacqary avatar Jul 16 '24 16:07 Zacqary

/ci

Zacqary avatar Jul 16 '24 20:07 Zacqary

/ci

Zacqary avatar Jul 17 '24 17:07 Zacqary

/ci

Zacqary avatar Jul 17 '24 19:07 Zacqary

A lot of test failures are related to the app state not storing properly in the URL. This might be a fairly involved refactor, I'll try to make it minimally invasive.

Zacqary avatar Jul 17 '24 22:07 Zacqary