search

OpenLiberty / liberty-tools-intellij

IntelliJ IDEA extension for Liberty
https://plugins.jetbrains.com/plugin/14856-open-liberty-tools
Eclipse Public License 2.0
11 stars 20 forks source link

Fix for Sending Notification to Slack Channel on List of PRs in LSP4IJ #837

Open vaisakhkannan opened 1 week ago

vaisakhkannan commented 1 week ago

Fixes https://github.com/OpenLiberty/liberty-tools-intellij/issues/803 Added a new YAML file, pr-slack-notification.yaml, for sending Slack notifications based on cron job runs.

Fixes https://github.com/OpenLiberty/liberty-tools-intellij/issues/810

At the end of pr-slack-notification.yaml, I have added one more job called slack-notification, which is used to trigger Slack Workflow Builder as a fix for the issue https://github.com/OpenLiberty/liberty-tools-intellij/issues/810

vaisakhkannan commented 1 week ago

When I checked during testing in the cron job runs, I saw that fetching the Closed PR list was correct, but it was failing to send the payload. When I tested it locally, it was working. So I made a change to send it as a single payload that can be used for sending both open and closed PR lists in a single step in the workflow file. I hope it will work in today's cron jobs.

aparnamichael commented 16 hours ago

@vaisakhkannan Can you please share the link of one workflow run?