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

Change graph workspace to store index pattern id instead of name #34989

Closed
flash1293 opened this issue Apr 12, 2019 · 3 comments · Fixed by #85295
Closed

Change graph workspace to store index pattern id instead of name #34989

flash1293 opened this issue Apr 12, 2019 · 3 comments · Fixed by #85295
Labels
Feature:Graph Graph application feature Feature:Saved Objects Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@flash1293
Copy link
Contributor

flash1293 commented Apr 12, 2019

Currently the graph-workspace saved object contains the name of the selected index pattern.
To be able to export and import them reliably (#34862), this should be changed to store the index pattern id instead (plus migration of saved workspaces).

To be able to do that, the migration has to support asynchronous steps (because it is necessary to look up the corresponding index pattern id during migration, issue for that is pending).

@flash1293 flash1293 added Feature:Graph Graph application feature Feature:Saved Objects Team:Visualizations Visualization editors, elastic-charts and infrastructure labels Apr 12, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app

@mikecote
Copy link
Contributor

Once this is completed, it will unblock #34997.

@timroes
Copy link
Contributor

timroes commented Apr 15, 2019

Blocked by #34996

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Graph Graph application feature Feature:Saved Objects Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants