[Cases] Use the bulk create attachment endpoint on the UI#130124
[Cases] Use the bulk create attachment endpoint on the UI#130124cnasikas merged 8 commits intoelastic:mainfrom
Conversation
|
Pinging @elastic/response-ops (Team:ResponseOps) |
|
Pinging @elastic/response-ops-cases (Feature:Cases) |
academo
left a comment
There was a problem hiding this comment.
I really liked how you made the change to be a drop-in replacement. Only missing a small validation to prevent a wrong API call.
Something I'd like to change but you don't have to because it will be a lot of work, is to remove the "bulk" from the name to simply createAttachments as the name already implies plural.
If you feel like doing it, go for it, but I won't hold the PR for that.
academo
left a comment
There was a problem hiding this comment.
Thanks for the renaming!
💛 Build succeeded, but was flakyTest Failures
Metrics [docs]Async chunks
Page load bundle
History
To update your PR or re-run it, just comment with: cc @cnasikas |
Fix mapping## Summary
This PR changes the UI to use the new bulk create attachment internal API.
Depends on: #129092
Checklist
Delete any items that are not applicable to this PR.
For maintainers