Conversation
|
@jmikell821 I've removed the following PRs that are not Security PRs but were pulled in by the release notes tool:
|
dplumlee
left a comment
There was a problem hiding this comment.
looks good on the bugs section, i didn't work too much on the features this release
|
I am not familiar with the release notes 🙂 so sorry if I am mistaken. I have some questions:
|
|
Hey @cnasikas, not a problem, and thanks for reviewing these! The Swimlane PR that was flagged for release notes had the To address your second question, our release notes tool didn't pull in any PRs for the RBAC for Cases feature. I did some quick digging to see where that PR might be and found elastic/kibana#95058. If this is the PR that should be release-noted for 7.14, it might've gotten overlooked by our tool because--although it has the appropriate team labels--the release notes label ( |
|
Ah thanks for the heads up about the labels @nastasha-solomon |
|
Thanks @nastasha-solomon for explaining the release note labels. It is difficult for me to understand which labels should I use every time. About the Swimlane connector, I think is more appropriate to be on the Feature section as it is a new feature. You are right about the RBAC. This is the right PR and we should put another release label also. I think RBAC should also be in the feature section because the user can create a role and set specific privileges (all/write/none) for Cases. It is a new capability that was not possible prior to 7.14. What do you think? |
joepeeples
left a comment
There was a problem hiding this comment.
Just a few small line-edits, looks good!
|
Hi @nastasha-solomon, could we please add a "Known Issue" for this bug. We could say something like: When upgrading from 7.8 to 7.14, rule execution and activation/deactivation of rules could fail (elastic/kibana#106233). To work around this issue you can use the PATCH rule API to update each rule that presents this problem. The payload of the PATCH call should set the |
peluja1012
left a comment
There was a problem hiding this comment.
LGTM. Thanks for including the last minute "known issue"!
* First draft. * Added features and enhancements. * Removed duplicates. * Saving changes. * Saving updates. * Saving changes. * Saving updates. * Saving changes. * Fixing broken link and adding more changes. * Saving updates. * Saving changes. * Adding two known issues. * Changes to case PRs. * Added Joe's comments. * Fixed typo. * Saving updates. * New styling for PATCH API. * Added Joe * Fixed punctuation errors.
* First draft. * Added features and enhancements. * Removed duplicates. * Saving changes. * Saving updates. * Saving changes. * Saving updates. * Saving changes. * Fixing broken link and adding more changes. * Saving updates. * Saving changes. * Adding two known issues. * Changes to case PRs. * Added Joe's comments. * Fixed typo. * Saving updates. * New styling for PATCH API. * Added Joe * Fixed punctuation errors.
* First draft. * Added features and enhancements. * Removed duplicates. * Saving changes. * Saving updates. * Saving changes. * Saving updates. * Saving changes. * Fixing broken link and adding more changes. * Saving updates. * Saving changes. * Adding two known issues. * Changes to case PRs. * Added Joe's comments. * Fixed typo. * Saving updates. * New styling for PATCH API. * Added Joe * Fixed punctuation errors.
* First draft. * Added features and enhancements. * Removed duplicates. * Saving changes. * Saving updates. * Saving changes. * Saving updates. * Saving changes. * Fixing broken link and adding more changes. * Saving updates. * Saving changes. * Adding two known issues. * Changes to case PRs. * Added Joe's comments. * Fixed typo. * Saving updates. * New styling for PATCH API. * Added Joe * Fixed punctuation errors.
Addresses #792.
Preview here.