Gliffy

Notify Watchers in Confluence to Changes in Gliffy Diagram

Details

  • Description:

    From Customer:
    We noticed that when a change is made to a gliffy diagram, that if you are watching the page or space in Confluence, no notification goes out. Are we doing something wrong? or is there a way to force a notification?

Issue Links

Activity

Hide
Chris Kohlhardt added a comment - 14/Mar/08 1:14 PM

Shannon, can you please investigate this?

Show
Chris Kohlhardt added a comment - 14/Mar/08 1:14 PM Shannon, can you please investigate this?
Hide
Shannon Krebs added a comment - 15/Mar/08 8:35 AM

We get a notification when a diagram is created because the gliffy macro is inserted into the page causing a page updated event. When an attachment is saved there is a AttachmentUpdatedEvent created in the AttachmentManager, but the NotificationListener only listens to add and remove attachment events, hence no notifications.

Looking at the code that does the update of attachments from the web interface, they trigger their own AttachmentCreatedEvent (even if its an update) so thats what we need to add to the gliffy code.

Unfortunately at some point Atlassian have moved the EventManager class into its own package, therefore this will need to be implemented as a versioned method because of the different package names

Show
Shannon Krebs added a comment - 15/Mar/08 8:35 AM We get a notification when a diagram is created because the gliffy macro is inserted into the page causing a page updated event. When an attachment is saved there is a AttachmentUpdatedEvent created in the AttachmentManager, but the NotificationListener only listens to add and remove attachment events, hence no notifications. Looking at the code that does the update of attachments from the web interface, they trigger their own AttachmentCreatedEvent (even if its an update) so thats what we need to add to the gliffy code. Unfortunately at some point Atlassian have moved the EventManager class into its own package, therefore this will need to be implemented as a versioned method because of the different package names
Hide
Shannon Krebs added a comment - 16/Mar/08 6:38 AM

Versioned methods implemented, tested locally on conf 2.7.

The email that get sent out for updates isn't really very helpful to users however, basically just alerts them that the attachment file has been updated, with a link directly to the attachment file.

Possibly in the future we should investigate if it is possible to override/extend/replace the default notification implementation somehow to provide a more relevant email.

Show
Shannon Krebs added a comment - 16/Mar/08 6:38 AM Versioned methods implemented, tested locally on conf 2.7. The email that get sent out for updates isn't really very helpful to users however, basically just alerts them that the attachment file has been updated, with a link directly to the attachment file. Possibly in the future we should investigate if it is possible to override/extend/replace the default notification implementation somehow to provide a more relevant email.

People

Dates

  • Created:
    14/Mar/08 12:19 PM
    Updated:
    18/Mar/10 6:31 PM
    Resolved:
    16/Mar/08 6:38 AM