mattermost / mattermost-marketplace

The stateless HTTP service backing the Mattermost marketplace.
http://integrations.mattermost.com
Apache License 2.0
38 stars 24 forks source link

[fix] CI Notification #452

Closed phoinixgrr closed 3 weeks ago

phoinixgrr commented 1 month ago

Summary

Inputing the output of mattermost.json to the PAYLOAD input of the notify action.

Builds on https://github.com/mattermost/mattermost-marketplace/pull/449#issuecomment-2286027219

image

Ticket Link

phoinixgrr commented 1 month ago

The changes look fine. Is there a way we can test them before merging?

Unfortunately not an easy one. Worst case scenario, notification will fail again. No other impact.

hanzei commented 1 month ago

@mgdelacroix Would you be the right person from ABC to review this PR?

mgdelacroix commented 3 weeks ago

@hanzei it seems that there is no specific knowledge around this in my team, so I'll do it