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
Event rule config enhancement #159
base: main
Are you sure you want to change the base?
Conversation
c40476a
to
8e16e7e
Compare
1c9054f
to
7ab548f
Compare
25b4a12
to
163112c
Compare
97018d5
to
4b1380c
Compare
2b31a5a
to
4bbe6f0
Compare
220a79d
to
c81fd09
Compare
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.
The delete button doesn't show any hint why it is deactivated. I can't find the previous comment, but I've requested this change before also.
$buttonsWrapper->add( | ||
[$eventRuleConfigSubmitButton, $discardChangesButton, $deleteButton] | ||
); | ||
$buttonsWrapper->add([$eventRuleConfigSubmitButton, $discardChangesButton, $deleteButton]); |
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.
This button can then be added conditionally.
This comment is present in PR #170. You are right, it must be fixed here. |
6a76ace
to
3dae615
Compare
8b5bffe
to
be66793
Compare
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.
Looks good to me, works fine.
@nilmerg please have a look.
Please squash the commits. |
be66793
to
d31e8a6
Compare
Please do not forget to keep an eye on the phpstan errors. |
d31e8a6
to
fd35468
Compare
Use a single form for event rule configuration.
Blocked by Icinga/icingaweb2#5190