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

[skip ci] prototype for saved objects relationships #27155

Closed

Conversation

mikecote
Copy link
Contributor

@mikecote mikecote commented Dec 13, 2018

do not merge

I wrote up a basic prototype of how we can save references into each saved objects and put them back into the object after reading.

This prototype is functional for dashboards, visualizations (from index pattern) and saved searches. What's left is visualizations from a saved search and test timelion / graph for index patterns.

See #27210.

@mikecote mikecote added WIP Work in progress Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc labels Dec 13, 2018
@mikecote mikecote self-assigned this Dec 13, 2018
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-platform

return this.hydrateIndexPattern();
}).then(() => {
injectReferences.call(this, resp.references);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This can probably be moved after parseSearchSource and implemented in a way that doesn't touch this.

panel.type = reference.type;
delete panel.panelReference;
});
this.panelsJSON = JSON.stringify(panels);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Probably have a way that doesn't modify this and works with arguments instead.

@mikecote mikecote force-pushed the saved-objects-relationships-prototype branch from ce1afed to 11f7e20 Compare December 14, 2018 15:30
@mikecote mikecote force-pushed the saved-objects-relationships-prototype branch from 11f7e20 to b0b9333 Compare December 14, 2018 19:44
@mikecote
Copy link
Contributor Author

mikecote commented Jan 7, 2019

Closing this in favour of the implementation PR (#28199) that is based from this.

@mikecote mikecote closed this Jan 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants