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.

Sign up for GitHub

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

Jump to bottom

[Sentinel] - Frequent Insufficient Funds Notifications in Sentinel #227

Closed
ngmachado opened this issue Jun 10, 2023 · 0 comments · Fixed by #229
Closed

[Sentinel] - Frequent Insufficient Funds Notifications in Sentinel #227

ngmachado opened this issue Jun 10, 2023 · 0 comments · Fixed by #229
Assignees
@ngmachado
Labels
Team: Protocol Protocol Core, Sentinel, Peripherals, Protocol Infrastructure Tools & DevOps

Comments

@ngmachado
Copy link
Collaborator

The current Sentinel version is generating insufficient funds notifications at a very high frequency, 1 notifications every 5 minute. This leads to a large amount of notifications that can be considered spammy and makes it hard for users to identify other potentially important notifications.
@ngmachado ngmachado added the Team: Protocol Protocol Core, Sentinel, Peripherals, Protocol Infrastructure Tools & DevOps label Jun 10, 2023
@ngmachado ngmachado self-assigned this Jun 10, 2023
@ngmachado ngmachado linked a pull request Jun 10, 2023 that will close this issue
Fix #227 - Reduce low balance alerts #229
Merged
@ngmachado ngmachado mentioned this issue Jun 10, 2023
Merged
@d10r d10r closed this as completed in #229 Jun 13, 2023
d10r pushed a commit that referenced this issue Jun 13, 2023
@ngmachado
Fix #227 - Reduce low balance alerts (#229)
eb9a755 
* add isAccountBalanceBelowMinimum to client

* add 12 hours interval to balance report

* if no threshold file, default to zero
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ngmachado ngmachado

Labels
Team: Protocol Protocol Core, Sentinel, Peripherals, Protocol Infrastructure Tools & DevOps
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #227 - Reduce low balance alerts superfluid-finance/superfluid-sentinel
1 participant
@ngmachado