fix: Closing PR's has side effects and is called often. Only notify … (…

…#950)

.github/workflows/notifications.yml

@@ -3,30 +3,14 @@ name: Notifications
 
 on:
   workflow_run: 
-    workflows: [PR,Merge]
+    workflows: [PR,PR Closed,Merge]
 jobs:
-  vars:
-    name: Set Variables
-    outputs:
-      pr: ${{ steps.pr.outputs.pr }}
-    runs-on: ubuntu-22.04
-    timeout-minutes: 1
-    steps:
-      # Get PR number for squash merges to main
-      - name: PR Number
-        id: pr
-        uses: bcgov-nr/[email protected]
-
   notify-teams:
     name: Notify Teams 
-    needs: [vars]
     runs-on: ubuntu-22.04
     steps:
       - name: MS Teams Notifications
         uses: dchourasia/[email protected]
         with:
           github-token: ${{ secrets.GITHUB_TOKEN }}
           webhook-uri: ${{ vars.MS_TEAMS_WEBHOOK_URI }}
-          custom-facts: |
-            - name: Notify
-              value: "PR# ${{ needs.vars.outputs.pr }}"