-
-
Notifications
You must be signed in to change notification settings - Fork 559
[Feature] Add Alerts History page #973
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
[Feature] Add Alerts History page #973
Conversation
|
Great idea! I'll check this out soon. How long do you think we should we retain these? 60 days? |
|
Great idea for the retention, I made it user configurable for 1,3,6,12 months. With default to 3 months. As well do you think the settings page is the right place for now? |
|
Thanks, I like this feature and want to get it in the 0.12.0 release so I'll pick it up from here. We'll just leave it in settings for now but I agree a sidebar would make sense in the future. |
|
Thanks again, this is merged now and is definitely a useful feature. I'm going update tests and docs for the next release, then I'll loop back around to the other PRs afterwards 👍 |
|
Thanks! Happy to be able to contribute to your amazing project! Take your time for the other ones. |
📃 Description
This PR introduces a new page to track the history of all alert results. For now, the page is located under the Settings section, as I could not find a better place for now.
🪵 Changelog
➕ Added
alert_historycollection to store alert result historyalert_historymodule to manage related logic📷 Screenshots