Add support for live activities#16046
Merged
Merged
Conversation
If the app supports live activities, and a particular live activity is available for this match, we want to allow the user to turn it on or off. If not, we want to check if match notifications are available instead, and show a button for that. If these aren't available either, the app might tell us the reason why (e.g. the user has already subscribed to notifications by following a team), and we show that information to the user. If none of these features are available, or we're not in the app at all, we show nothing. The majority of this change involves refactoring the `Notifications` sub-component of `FootballMatchHeader` to support this, and updating the stories. **Note:** For now this change is using a pre-release version of Bridget and hardcodes the expected new Bridget version for testing. This will be updated to use the production release of Bridget once it's available.
alexduf
approved these changes
Jun 4, 2026
alexduf
left a comment
alexduf
left a comment
Contributor
There was a problem hiding this comment.
Nice, thanks for guiding us through the code 👍
jamesmockett
reviewed
Jun 4, 2026
jamesmockett
reviewed
Jun 4, 2026
jamesmockett
approved these changes
Jun 4, 2026
jamesmockett
left a comment
jamesmockett
left a comment
Contributor
There was a problem hiding this comment.
Thanks for the walkthrough, it was really helpful. This all looks good 👍
The new version includes the `LiveActivities` service.
|
Hello 👋! When you're ready to run Chromatic, please apply the You will need to reapply the label each time you want to run Chromatic. |
|
Seen on PROD (merged by @JamieB-gu 27 minutes and 25 seconds ago) Please check your changes! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
If the app supports live activities, and a particular live activity is available for this match, we want to allow the user to turn it on or off. If not, we want to check if match notifications are available instead, and show a button for that. If these aren't available either, the app might tell us the reason why (e.g. the user has already subscribed to notifications by following a team), and we show that information to the user. If none of these features are available, or we're not in the app at all, we show nothing.
The majority of this change involves refactoring the
Notificationssub-component ofFootballMatchHeaderto support this, and updating the stories.Part of guardian/bridget#252
Screenshots
Live Activities
If live activities are supported.
Notifications
If live activities aren't supported, but notifications are.
Unavailable With Reason
Live activities aren't available, and notifications aren't available because you follow at least one of the teams already.
Unavailable
Neither live activities nor notifications are available, and we have no explanation as to why.