-
Notifications
You must be signed in to change notification settings - Fork 66
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
Add support for archive and delete on report instances #170
Comments
Thanks @camAtGitHub, @opensearch-project/admin can you transfer this issue to https://github.com/opensearch-project/dashboards-reports |
The current implementation doesn't allow delete report instances from UI. The original purpose is to keep every report(not report definition) as audit trail. But I think the reported issue make sense for a real use case. I am thinking we improve the UX by adding additional checks to see if the saved objects(searches, dashboards, visualiztions) exist, before rendering report details page. If not exist, we show an alert to user. Basically we make sure user doesn't end up with a broken url, not being aware of what was the issue. @kgcreative any thoughts on this? |
relabel it , since this is not a bug, but a feature request |
@zhongnansu - I think there are valid use cases where we might want to delete reports, either because someone created test reports they want to clean up, or becuase the underlying data has been archived, rolled over, or deleted and the report no longer can be generated. Let's work together on improving this. |
Describe the bug
Can't interact / delete / archive reports in the report plugin.
Deleted reports/searches have broken links.
To Reproduce
Steps to reproduce the behavior:
Note the line item of the on_demand_report
All links for the 'on_demand_report' are broken.
Expected behavior
Ability to delete 'reports' as listed in the 'reports' section.
Same goes for reports with a source of 'Notebook'
OpenSearch Version
1.0.1
Dashboards Version
1.0.1
Plugins
Standard as per docker 1.0.1 image.
Screenshots
The text was updated successfully, but these errors were encountered: