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

[dev.icinga.com #2636] Function 'Delete comments' in icinga-web #790

Closed
icinga-migration opened this issue May 22, 2012 · 11 comments
Labels
Milestone

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented May 22, 2012

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

Created by ossmon on 2012-05-22 12:10:14 +00:00

Assignee: mhein
Status: Resolved (closed on 2013-10-16 14:45:10 +00:00)
Target Version: 1.10
Last Update: 2013-10-16 14:45:30 +00:00 (in Redmine)


In the Icinga-Classic Gui we have the possibility to delete comments.
This feature doesn't exist in the icinga-web.

Feature request was already done in 1.6. See #1438, but it doesn't work in the tackle as described in #1438.

Because the tackle is experimental, i think it would be perhaps a better idea to add this feature as a command "delete a host/service comment" as it already exists a command "add a host/service command" in the 'commands" list box

CommandButton.JPG

Attachments

Changesets

2013-10-09 08:16:08 +00:00 by mhein 36d3da7

Add del all comments to objects

resolves #2636

2013-10-19 14:51:03 +00:00 by (unknown) 09ab47c

Fix DEL_ALL_SVC_COMMENTS doesn't work.

fixes #4900
refs #2636

Relations:

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jul 3, 2012

Updated by ossmon on 2012-07-03 07:13:31 +00:00

Icinga-web 1.7:

i found following records about the deletion of comments in xml files in ..icinga-web/app/modules/Api/config/commands:

host.xml:
    
        DEL_HOST_COMMENT
        host
        icinga-icon-bricks
        Remove comment
        
            
        
    

notification.xml:
    
        DEL_SVC_COMMENT
        notification
        icinga-icon-bricks
        Delete comment

This commands are not shown on icinga-web. Is it only a preparation for a future implementation or a bug ?

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Sep 24, 2012

Updated by mfriedrich on 2012-09-24 13:34:41 +00:00

for deleting comments and acks you'll always need the id - i guess that's the missing bit for now.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 8, 2013

Updated by mfriedrich on 2013-04-08 20:34:47 +00:00

  • Status changed from New to Assigned
  • Assigned to set to mhein

@marius

please verify and check for a possible fix.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Oct 4, 2013

Updated by adrianlzt on 2013-10-04 09:55:28 +00:00

Any news about this?

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Oct 4, 2013

Updated by mhein on 2013-10-04 09:59:01 +00:00

Hi,

problem here is that we do not have a reference for a single comment in this grid. What we can implement is a "DEL_ALL_HOST_COMMENTS" and "DEL_ALL_SVC_COMMENTS" quickly.

Kind regards,
Marius.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Oct 4, 2013

Updated by ossmon on 2013-10-04 11:34:27 +00:00

It will be a first good step if you do it.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Oct 9, 2013

Updated by mhein on 2013-10-09 07:55:48 +00:00

  • Target Version set to 1.10
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Oct 9, 2013

Updated by mhein on 2013-10-09 08:20:29 +00:00

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

Applied in changeset 36d3da7.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Oct 9, 2013

Updated by mhein on 2013-10-09 08:21:13 +00:00

  • Status changed from Resolved to Assigned
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Oct 9, 2013

Updated by mhein on 2013-10-09 08:21:22 +00:00

  • Status changed from Assigned to 8
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Oct 16, 2013

Updated by mfrosch on 2013-10-16 14:45:10 +00:00

  • Status changed from 8 to Resolved
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
1 participant
You can’t perform that action at this time.