Skip to content

[7.x] [Core] Move Saved objects files to core (#38771)#39158

Merged
rudolf merged 2 commits intoelastic:7.xfrom
rudolf:backport/7.x/pr-38771
Jun 19, 2019
Merged

[7.x] [Core] Move Saved objects files to core (#38771)#39158
rudolf merged 2 commits intoelastic:7.xfrom
rudolf:backport/7.x/pr-38771

Conversation

@rudolf
Copy link
Copy Markdown
Contributor

@rudolf rudolf commented Jun 18, 2019

Backports the following commits to 7.x:

@rudolf rudolf added the backport This PR is a backport of another PR label Jun 18, 2019
@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

* Move src/legacy/server/saved_objects -> src/core/server/saved_objects

* Fix SavedObject import references after moving files to core

* First pass at SavedObjects api docs

* Expose and import all saved object types through core/server

* Don't expose SavedObjectsManagement from core and fix imports

* Improve typings for SavedObject error helpers

* Fix type errors after master merge

* Fix SavedObjectErrorHelpers tests
@rudolf rudolf force-pushed the backport/7.x/pr-38771 branch from c99453c to 0f456f5 Compare June 18, 2019 13:28
@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@rudolf
Copy link
Copy Markdown
Contributor Author

rudolf commented Jun 18, 2019

retest

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@rudolf
Copy link
Copy Markdown
Contributor Author

rudolf commented Jun 18, 2019

retest

1 similar comment
@rudolf
Copy link
Copy Markdown
Contributor Author

rudolf commented Jun 19, 2019

retest

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@legrego
Copy link
Copy Markdown
Member

legrego commented Jun 19, 2019

retest

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@rudolf rudolf merged commit 986b70c into elastic:7.x Jun 19, 2019
@rudolf
Copy link
Copy Markdown
Contributor Author

rudolf commented Jun 19, 2019

💚 Build Succeeded

@legrego you seem to have the magic touch! 😮

@rudolf rudolf deleted the backport/7.x/pr-38771 branch June 19, 2019 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants