Skip to content

Conversation

@dsibilio
Copy link
Contributor

Checklist

Description

Now that all repositories have been migrated from using the send-teams-notification action with legacy webhooks to workflow-based webhooks, we can safely upgrade to skitionek/notify-microsoft-teams v1.0.9 instead of an older master-pinned version.

This solves the issue of jobs sending notifications to workflow-based webhooks being marked as failed on GHA although the notification is produced successfully, tested here: https://github.com/HylandExperience/studio-component-registry/actions/runs/15320185322/job/43102050236?pr=353#step:2:1

Copilot AI review requested due to automatic review settings May 29, 2025 09:02
@dsibilio dsibilio requested a review from a team as a code owner May 29, 2025 09:02
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR bumps the version of the Alfresco build tools and updates the usage of skitionek/notify-microsoft-teams to v1.0.9 to fix an issue with notification failures on workflow-based webhooks.

  • Update version in version.txt from v8.22.0 to v8.22.1.
  • Update the commit reference in the action.yml file for skitionek/notify-microsoft-teams to the new v1.0.9 release.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
version.txt Bumped the version for the release update
.github/actions/send-teams-notification/action.yml Updated the commit hash to reference skitionek/notify-microsoft-teams v1.0.9

Copy link
Contributor

@pmacius pmacius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good :)

@dsibilio dsibilio merged commit 6d729e1 into master May 29, 2025
7 checks passed
@dsibilio dsibilio deleted the fix/AAE-35342_bump-skitionek-action branch May 29, 2025 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants