-
Notifications
You must be signed in to change notification settings - Fork 202
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
Improve ActivityLog for services and servicesets #1816
Conversation
4050fff
to
cb4c50e
Compare
This was mainly a problem for IcingaServiceSet and others that may have a capital letter where the table name is split by `_`
cb4c50e
to
db00f1b
Compare
@Thomas-Gelf: Could you have a look at this? I'd be great to merge this soon. Should have no functional difference in terms of config, only activity log and visibility for diffs. |
* A header is now always rendered and includes assign and description * Diff is now visible for sets
* Service to Host or ServiceSet * Set to Host
5cac698
to
6d4475d
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.
LGTM, thank you! Please address the phpcs complaint regarding the missing space on IcingaServiceSet.php:379 (https://travis-ci.org/Icinga/icingaweb2-module-director/jobs/518241290#L569), then this should be ready for merge
Wait I add one more change |
I didn't re-check the whole diff, but the phpcs complaint should be fixed. Nice, go! |
assign_filter