-
Notifications
You must be signed in to change notification settings - Fork 8k
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
[RAM] allow user to predefined their id when they create connector #155392
Conversation
Pinging @elastic/response-ops (Team:ResponseOps) |
I spoke with @XavierM offline, the one change to implement in this PR is an error on create if ever the |
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.
Tested locally and changes LGTM! We talked about adding a unit test for the error when a preconfigured connector exists with the same id.
💚 Build Succeeded
Metrics [docs]Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: |
…lastic#155392) ## Summary allow user to predefined their id when they create connector ### Checklist - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios
Summary
allow user to predefined their id when they create connector
Checklist