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

Error loading alert condition details page #3534

Closed
edmundoa opened this issue Feb 22, 2017 · 1 comment
Closed

Error loading alert condition details page #3534

edmundoa opened this issue Feb 22, 2017 · 1 comment
Assignees
@dennisoelkers @edmundoa
Labels
bug web
Milestone
2.2.2

Comments

@edmundoa
Copy link
Contributor

edmundoa commented Feb 22, 2017 •
edited

Sometimes loading the alert condition details page only works partially, and the alert notifications associated to the stream are always in a loading state. The error in the console is:

Unhandled rejection TypeError: Cannot read property 'org.graylog2.alarmcallbacks.HTTPAlarmCallback' of undefined
    at Constructor.render (webpack:///./src/components/alertnotifications/AlertNotification.jsx?:92:59)

Expected Behavior

Alert notifications should load.

Current Behavior

Alert notifications do not always load properly.

Steps to Reproduce (for bugs)

  1. Go to Alerts -> Alert conditions -> Select an alert
  2. Check if the notifications appear (this is a timing issue, so it may not always happen)

Your Environment

  • Graylog Version: Graylog 2.3.0-SNAPSHOT, 2.2.1 is also affected
@edmundoa edmundoa self-assigned this Feb 22, 2017
@edmundoa edmundoa added this to the 2.2.2 milestone Feb 22, 2017
edmundoa pushed a commit that referenced this issue Feb 22, 2017
Wait until types of notifications are loaded
d9b8a56 
Fixes #3534
@edmundoa
Copy link
Contributor Author

Update: Updated the original issue report with some steps to reproduce and confirming that 2.2.1 is affected.

@edmundoa edmundoa mentioned this issue Feb 22, 2017
Merged
edmundoa pushed a commit that referenced this issue Feb 23, 2017
Wait until notification types are loaded
e18d711 
Fixes #3534
@ghost ghost assigned dennisoelkers Feb 23, 2017
@ghost ghost removed the in progress label Feb 23, 2017
dennisoelkers pushed a commit that referenced this issue Feb 23, 2017
@edmundoa @dennisoelkers
Wait until notification types are loaded (#3537)
0a3b881 
Fixes #3534
dennisoelkers pushed a commit that referenced this issue Feb 23, 2017
@edmundoa @dennisoelkers
Wait until notification types are loaded (#3537)
0e5e7f6 
Fixes #3534
(cherry picked from commit 0a3b881)
@dennisoelkers dennisoelkers mentioned this issue Feb 23, 2017
Merged
kroepke pushed a commit that referenced this issue Feb 23, 2017
@dennisoelkers @kroepke
Wait until notification types are loaded (#3537) (#3542)
7cec8c5 
Fixes #3534
(cherry picked from commit 0a3b881)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dennisoelkers dennisoelkers

@edmundoa edmundoa

Labels
bug web
Projects
None yet
Milestone
2.2.2
Development

No branches or pull requests
2 participants
@dennisoelkers @edmundoa