Skip to content
This repository was archived by the owner on Jul 15, 2025. It is now read-only.

LIBAAEC-54 Send teams notification on pr#494

Merged
hortongn merged 18 commits intoqafrom
send-teams-notification-on-pr
Apr 7, 2025
Merged

LIBAAEC-54 Send teams notification on pr#494
hortongn merged 18 commits intoqafrom
send-teams-notification-on-pr

Conversation

@Janell-Huyck
Copy link
Contributor

@Janell-Huyck Janell-Huyck commented Apr 7, 2025

Jira issue: #54
This is an experiment to see if we can get Github to post to Teams when a new PR is created. It uses an "incoming webhook" integration with Teams, where Teams generated an URL for the webhook to use.

The URL is stored in Github as a "repository secret" with the name TEAMS_WEBHOOK_URL. When we migrate to git.uc.edu that repository secret will need to be copied forward. I expect the integration to continue to work since git.uc.edu can run webhooks, and Teams doesn't need to care who is posting to that webhook URL.

@Janell-Huyck Janell-Huyck changed the title Send teams notification on pr HOLD UNTIL NEXT DEPLOY - Send teams notification on pr Apr 7, 2025
@Janell-Huyck Janell-Huyck added the Do Not Merge This PR request should not be merged. label Apr 7, 2025
@Janell-Huyck Janell-Huyck changed the title HOLD UNTIL NEXT DEPLOY - Send teams notification on pr HOLD UNTIL NEXT DEPLOY - LIBAAEC-54 Send teams notification on pr Apr 7, 2025
@Janell-Huyck Janell-Huyck reopened this Apr 7, 2025
@Janell-Huyck Janell-Huyck reopened this Apr 7, 2025
@Janell-Huyck Janell-Huyck reopened this Apr 7, 2025
@Janell-Huyck Janell-Huyck reopened this Apr 7, 2025
@Janell-Huyck Janell-Huyck reopened this Apr 7, 2025
@Janell-Huyck Janell-Huyck reopened this Apr 7, 2025
@Janell-Huyck Janell-Huyck reopened this Apr 7, 2025
@Janell-Huyck Janell-Huyck reopened this Apr 7, 2025
@Janell-Huyck Janell-Huyck reopened this Apr 7, 2025
@Janell-Huyck Janell-Huyck reopened this Apr 7, 2025
@Janell-Huyck Janell-Huyck reopened this Apr 7, 2025
@Janell-Huyck Janell-Huyck reopened this Apr 7, 2025
@Janell-Huyck Janell-Huyck reopened this Apr 7, 2025
@Janell-Huyck Janell-Huyck reopened this Apr 7, 2025
@Janell-Huyck Janell-Huyck changed the title HOLD UNTIL NEXT DEPLOY - LIBAAEC-54 Send teams notification on pr LIBAAEC-54 Send teams notification on pr Apr 7, 2025
@Janell-Huyck Janell-Huyck removed the Do Not Merge This PR request should not be merged. label Apr 7, 2025
@hortongn hortongn merged commit 4caef2c into qa Apr 7, 2025
11 checks passed
@hortongn hortongn deleted the send-teams-notification-on-pr branch April 7, 2025 20:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants