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

[NP] Migrate saved object mappings and migration scripts for Dashboard #63864

Merged
merged 10 commits into from
Apr 22, 2020

Conversation

maryia-lapata
Copy link
Contributor

@maryia-lapata maryia-lapata commented Apr 17, 2020

Summary

Part of #61259.

Move the dashboard saved object type registration from the legacy world to Kibana platform.

@maryia-lapata maryia-lapata added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Feature:NP Migration v7.8.0 labels Apr 17, 2020
@maryia-lapata
Copy link
Contributor Author

@elasticmachine merge upstream

@maryia-lapata maryia-lapata marked this pull request as ready for review April 22, 2020 06:29
@maryia-lapata maryia-lapata requested a review from a team April 22, 2020 06:29
@maryia-lapata maryia-lapata requested a review from a team as a code owner April 22, 2020 06:29
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

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

Copy link
Contributor

@pgayvallet pgayvallet left a comment

Choose a reason for hiding this comment

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

LGTM for platform changes.

Referencing #63943 as I will have to integrate these changes

@@ -230,6 +230,7 @@ export {
SavedObjectsMigrationLogger,
SavedObjectsRawDoc,
SavedObjectSanitizedDoc,
SavedObjectUnsanitizedDoc,
Copy link
Contributor

Choose a reason for hiding this comment

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

@rudolf I'm going to stop fighting against this export. It's a parameter type of a public core API anyway.

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 LGTM, dashboards still save and load. I exported a dashboard in 6.8 and imported it here to test the migration - this worked fine as well except for a thrown exception that also appears on master, I will create a separate issue for this.

@maryia-lapata maryia-lapata merged commit 7969382 into elastic:master Apr 22, 2020
@maryia-lapata maryia-lapata deleted the dashboard-so branch April 22, 2020 14:33
maryia-lapata added a commit that referenced this pull request Apr 23, 2020
#63864) (#64194)

* Migrate saved objects

* Move common used types and fucntions to dashboard/common

* Fix unit tests

* Update server.api.md

* Fix TS

* Update i18n IDs

* Update TS

Co-authored-by: Elastic Machine <[email protected]>

Co-authored-by: Elastic Machine <[email protected]>
@maryia-lapata maryia-lapata added backported Team:Visualizations Visualization editors, elastic-charts and infrastructure labels Apr 23, 2020
@elasticmachine
Copy link
Contributor

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

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

Successfully merging this pull request may close these issues.

5 participants