fix: make "Check TestFlight" text a clickable link in Slack RC notifications#31018
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
baptiste-marchand
left a comment
There was a problem hiding this comment.
approving to unblock engagement CO requirement
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection: Performance Test Selection: |
Description
Make "Check TestFlight" text a clickable link in Slack RC notifications.
Changelog
CHANGELOG entry: null
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Performance checks (if applicable)
trace()for usage andaddTokenfor an exampleFor performance guidelines and tooling, see the Performance Guide.
Pre-merge reviewer checklist
Note
Low Risk
Only changes fallback Slack mrkdwn text in a non-blocking RC notification script; no app runtime, auth, or data handling impact.
Overview
When the RC Slack notification has no valid iOS download URL, the iOS row no longer shows plain italic “Check TestFlight”. It now uses a Slack mrkdwn link to the public TestFlight join URL (
https://testflight.apple.com/join/hBrjtFuA), matching how Android shows a clickable “Download” when a URL exists.Reviewed by Cursor Bugbot for commit 5b2a2a5. Bugbot is set up for automated code reviews on this repo. Configure here.