Skip to content
This repository has been archived by the owner. It is now read-only.

[dev.icinga.com #2927] cgiparams: cmd.cgi: send_notifications and sticky_ack now require 0|1 values, default is enabled #332

Closed
icinga-migration opened this issue Aug 1, 2012 · 4 comments

Comments

Projects
None yet
1 participant
@icinga-migration
Copy link
Member

commented Aug 1, 2012

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

Created by mfriedrich on 2012-08-01 17:16:57 +00:00

Assignee: croft
Status: Resolved (closed on 2012-08-29 20:21:48 +00:00)
Target Version: 1.8
Last Update: 2012-08-29 20:21:48 +00:00 (in Redmine)

Icinga Version: 1.7.1

this change was necessary to honor the default CHECKED checkbox on the cmd.cgi, letting that being overridden correctly by the GET params.

  • send_notification=0|1, default is enabled
  • sticky_ack=0|1, default is enabled

see #2926 for more info.


Relations:

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Aug 26, 2012

Updated by croft on 2012-08-26 20:32:11 +00:00

  • Status changed from New to Assigned
  • Assigned to set to croft
  • Icinga Version set to 1
  • (unknown custom field) set to Linux
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Aug 26, 2012

Updated by croft on 2012-08-26 21:19:45 +00:00

  • Status changed from Assigned to Feedback
  • Done % changed from 0 to 50

Hmm, I just added the possible values to the table in cgiparams.xml, is this enough or more explanations somewhere?

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Aug 27, 2012

Updated by mfriedrich on 2012-08-27 07:58:02 +00:00

a short note on the change would be nice, details can be found within the core's Changelog CHANGES section.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Aug 29, 2012

Updated by croft on 2012-08-29 20:21:48 +00:00

  • Status changed from Feedback to Resolved
  • Done % changed from 50 to 100

Done.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.