-
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
[Managed content] Relax delete restrictions #179113
[Managed content] Relax delete restrictions #179113
Conversation
/ci |
/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.
👍
@elasticmachine merge upstream |
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.
Overall lgtm! Just a thought: would it be feasible to give integrations users a way to select the subset of assets they want rather than disabling this feature?
Did not test locally this time. Approving to unblock!
cc @teresaalvarezsoler who will be the PM for content management stuff moving forward. For my part, I think something more like your suggestion could be an elegant solution. But, it involves more coordination with Fleet and product definition effort when I'm really just trying to get this project off my back ;) At least allowing users to delete assets is no worse a situation than we were in before. |
Also, pinging @amyjtechwriter — we are removing delete restrictions in 8.14 at @teresaalvarezsoler 's request. The docs should be updated for that release. Let me know if you want me to create an issue somewhere. |
💛 Build succeeded, but was flaky
Failed CI StepsMetrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
Summary
Relaxes delete restrictions for managed content on both the library views and SO management.
Close #179099.
Some managed saved objects for testing: