-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Content Management] Server side client #175968
[Content Management] Server side client #175968
Conversation
/ci |
c3e07b2
to
73a0d61
Compare
/ci |
Pinging @elastic/appex-sharedux (Team:SharedUX) |
Pinging @mattkime @thomasneirynck @davismcphee as you all requested this feature to be released at some point. |
/ci |
@elasticmachine run elasticsearch-ci/docs |
/ci |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, very straightforward CRUD additions
storageContext: StorageContext; | ||
} | ||
|
||
const secretToken = Symbol('secretToken'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note to self: this never gets stored. I had to check because that could pose an issue if we ever handle a token that was created in a separate Kibana instance
Thanks for the review @tsullivan ! |
💚 Build Succeeded
Metrics [docs]Public APIs missing comments
History
To update your PR or re-run it, just comment with: cc @sebelga |
In this PR I'v added a server side client for Content management.
The client operates on a single content type and has the same methods as the content storage:
get
bulkGet
create
update
delete
search
I've also exposed an multi type search client with a single method to search accros different content types.
Currently it only support searching content types stored using the Saved object client.
msearch
API
When registering a content type
From the plugin
setup
contractNotes on implementation
I've replaced the code in all the RPC procedures to consume the clients. This means that appart from the unit tests that I've added, the server side client is used on all the functional tests for the content registered in CM.
Fixes #154836
Fixes #170877