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

[dev.icinga.com #13157] Notification apply rule broken #599

Closed
icinga-migration opened this issue Nov 11, 2016 · 5 comments
Closed

[dev.icinga.com #13157] Notification apply rule broken #599

icinga-migration opened this issue Nov 11, 2016 · 5 comments
Labels
Milestone

Comments

@icinga-migration
Copy link

This issue has been migrated from Redmine: https://dev.icinga.com/issues/13157

Created by kevinh on 2016-11-11 06:04:37 +00:00

Assignee: (none)
Status: Resolved (closed on 2016-12-14 14:10:04 +00:00)
Target Version: 1.3.0
Last Update: 2016-12-14 14:10:04 +00:00 (in Redmine)


Hello,
as explained in
https://monitoring-portal.org/index.php?thread/38401-notification-apply-rule-error/

I am not able to create apply rules for notifications.
Everytime I try it, it will just open the dialog for creating a new notification.

Attachments

Changesets

2016-12-14 14:04:38 +00:00 by tgelf ec0ea2e

NotificationsController: use new base class...

...with less tabs, less and simpler code

fixes #13565
fixes #13157
@icinga-migration
Copy link
Author

Updated by tgelf on 2016-11-11 08:28:31 +00:00

Seems that something got weird with the passed parameters. Could you please let me know whether it works when you choose the desired template ("test" on your screenshot) in the right form ("Importe")? That should be preselected, but somehow got lost...

@icinga-migration
Copy link
Author

Updated by kevinh on 2016-11-11 08:43:35 +00:00

I tried what you said and it works. thanks for the workaround, hope the bug will be fixed soon :)

Edit: it does look weird though, is there any chance to make it look like the services with apply rules?

@icinga-migration
Copy link
Author

Updated by kevinh on 2016-11-11 08:47:19 +00:00

  • File added director_notification.PNG

@icinga-migration
Copy link
Author

Updated by tgelf on 2016-12-13 10:57:47 +00:00

  • Target Version set to 1.3.0

@icinga-migration
Copy link
Author

Updated by tgelf on 2016-12-14 14:10:04 +00:00

  • Status changed from New to Resolved
  • Done % changed from 0 to 100

Applied in changeset ec0ea2e.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant