search

datalust / seq-app-opsgenie

Create Opsgenie alerts in response to events or notifications in Seq
Apache License 2.0
3 stars 2 forks source link

Update description to InputType.LongText #8

Closed MattMofDoom closed 3 years ago

MattMofDoom commented 3 years ago

Hey @nblumhardt ,

I'll have to do PIV testing to double check that this works with the Opsgenie REST API, but when integrating Handlebars.NET to Seq.App.Atlassian.Jira, I noted that the Jira REST API integration worked better (in terms of formatting) if I pivoted the Description field to an InputType.LongText. Prior to that, functionality like headers and tables were problematic in the InputType.Text constraints, thanks to some of the interaction between Seq, the Jira app, and the Jira REST API.

Obviously the other benefit of this is that it's easier to see and edit the Handlebars template for description, which can potentially be quite long.

I've blogged about how Handlebars can be used with the Jira (and Opsgenie) apps and how truly useful it is at https://mattmofdoom.com/using-handlebars-templates-with-seqappatlassianjira-and-seqappopsgenie/

I took an earlier pass at whether there was anything else that could usefully be added to the OpsGenie app from all the interoperability work of late, but I think we're more or less getting to the feature complete stage for that v1.0 PR you have flagged 😁

Cheers mate,

Matt

nblumhardt commented 3 years ago

Looks good to me, thanks Matt! 👍