search

itomig-de / itomig-webhook-integration

3 stars 2 forks source link

New Feature: Add default notification text #6

Closed Hipska closed 5 years ago

Hipska commented 5 years ago

What do you think if basic text is left empty (or with option) to add a default text:

Object R-00001 is created.

Or

Object R-00001 has been updated from portal.

Or even

User Request R-00001 status changed to Assigned.

Text could be saved in dictionary, for example: class:TriggerOnStateEnter/Notification = %1$s %2$s status changed to <b>%3$s</b>.

  1. Class translation
  2. Object hyperlink
  3. More info

With this, there is no need to create a separate action anymore for every type of trigger or for every type of object you want to make.

What are your thoughts on this?

larhip commented 5 years ago

Maybe this is something which is interesting to discuss with combodo directly. maybe this could be integrated in the core of itop with a new type of placeholder. The gives the opportunity to use such text messages not just in the notification type but also in email notifications for example.

Hipska commented 5 years ago

Yes, good idea but that will tak time.

Couldn't we do it already in the meantime?

larhip commented 5 years ago

Hi @Hipska , I discussed this with a colleague yesterday and we decided to not implement this in this extension. From our point of view this feature definitely belongs to the implementation of the trigger and not of the actions. So I will close this issue. Anyhow, thank you a lot for your proposal! Best Lars

Hipska commented 5 years ago

Okay, good point.

Hipska commented 3 years ago

@Molkobain, what do you think of this? This would even be interesting to be used as title for email notifications..

Molkobain commented 3 years ago

Hello @Hipska ,

I'm not sure to understand where this would be used? That would be a default text to be used by the action in the body (for email actions) or payload (for webhook actions) ?

Idea seems interesting indeed! This would allow easier/faster notification configuration for the basic ones. @v-dumas what do you think?

Hipska commented 3 years ago

Yes indeed. I was thinking to make it available as a placeholder