-
Notifications
You must be signed in to change notification settings - Fork 0
Support host.vars.* and service.vars.* filters
#390
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
base: main
Are you sure you want to change the base?
Conversation
sukhwinder33445
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
Should be solved with Icinga/ipl-orm#152 |
Yes, it works. It now depends on the linked PR to work correctly. |
sukhwinder33445
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
library/Notifications/Web/Control/SearchBar/ObjectSuggestions.php
Outdated
Show resolved
Hide resolved
1465b7d to
a831648
Compare
a831648 to
43dfc97
Compare
|
Decided to render variables properly in an incident's detail. Also, array variables were incorrectly enriched after using them in the searchbar. |
43dfc97 to
f94efcd
Compare
f94efcd to
7906af3
Compare

requires Icinga/icinga-notifications#324 and Icinga/icingadb#998