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.

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

Error Notification Displayed When Unacknowledged Alert Limit Set to -1 #3313

Open
mastercactapus opened this issue Sep 27, 2023 · 0 comments · May be fixed by #4257
Open

Error Notification Displayed When Unacknowledged Alert Limit Set to -1 #3313

mastercactapus opened this issue Sep 27, 2023 · 0 comments · May be fixed by #4257
Labels
bug Something isn't working

Comments

@mastercactapus
Copy link
Member

Describe the Bug:
When setting the unacknowledged alert limit to -1 in the System Limits settings, the system erroneously displays an alert limit reached notification. Though the functionality of allowing unlimited alerts with -1 works correctly, the user interface persists in displaying the error notification.

Steps to Reproduce:

  1. Go to 'Admin'.
  2. Click on 'System Limits'.
  3. Set 'Unacked Alert Limit' to -1.
  4. Navigate to any service, observe the error notification despite it should indicate unlimited alerts due to the -1 setting.

Expected Behavior:
When setting the alert limit to -1, the user interface should not show an alert limit exceeded notification.

Observed Behavior:
Despite setting the alert limit to -1, an alert limit exceeded notification is displayed in the user interface.

Application Version:
This issue is not version or browser specific, it affects all versions and browsers.

Additional Context:
The bug seems to stem from the user interface or warning/notification code performing a count >= limit check without taking the -1 value into account.

@mastercactapus mastercactapus added the bug Something isn't working label Sep 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant