fix slack notification (#9146)

* fix slack notification * removed unnecessary checks
pixiebrix · Sep 12, 2024 · 58b8e49 · 58b8e49
1 parent c5a2652
commit 58b8e49
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/.github/workflows/e2e-test-pre-release-browsers.yml b/.github/workflows/e2e-test-pre-release-browsers.yml
@@ -18,13 +18,10 @@ jobs:
   slack-notification:
     needs: end-to-end-tests
     runs-on: ubuntu-latest
-    if: ${{ job.status == 'failure' }}
+    if: ${{ always() && (needs.end-to-end-tests.result == 'failure') }}
     steps:
       - name: Slack Notification
         uses: rtCamp/action-slack-notify@v2
-        with:
-          status: ${{ job.status }}
-          notify_when: "failure"
         env:
           SLACK_WEBHOOK: ${{ secrets.SLACK_WEBHOOK }}
           SLACK_CHANNEL: playwright-playground