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

[7.x] [Lists][Exceptions] - Updates exception list item comments structure (#68864) #68907

Merged
merged 2 commits into from
Jun 12, 2020

Conversation

yctercero
Copy link
Contributor

@yctercero yctercero commented Jun 11, 2020

Backports the following commits to 7.x:

API Change - Updates lists plugin exception list item structure of comment from string to array of objects.

…lastic#68864)

### Summary

This is part of a series of upcoming changes to the exception list item structure. This PR focuses solely on updating exception_item.comment. The hope is to keep these PRs relatively small.

- Updates exception_item.comment structure which was previously a string to exception_item.comments which is an array of { comment: string; created_by: string; created_at: string; }
- Adds a few unit tests server side
- Fixes some minor misspellings
- Updates ExceptionViewer component in the UI to account for new structure
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

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

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app-arch (Team:AppArch)

@yctercero yctercero merged commit 4304c4f into elastic:7.x Jun 12, 2020
@yctercero
Copy link
Contributor Author

NOTE: Had member of arch team take a look for a thumbs up.

@yctercero yctercero deleted the backport/7.x/pr-68864 branch October 14, 2020 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants