Skip to content

[DRAFT] Handle VAR reversals#1855

Draft
vlbee wants to merge 14 commits into
mainfrom
live/var-reversal
Draft

[DRAFT] Handle VAR reversals#1855
vlbee wants to merge 14 commits into
mainfrom
live/var-reversal

Conversation

@vlbee

@vlbee vlbee commented Jul 14, 2026

Copy link
Copy Markdown
Member

This is a WIP PR and will be replaced by smaller sequential prs.

What does this change?

Introduction of VAR in football matches means key events such as a goal or red card can be reversed. This is not captured in the incoming PA data; the overruled event simply disappears from the match event timeline. In practice this means LiveActivities and updated Android push notifications display the wrong score until a subsequent "triggering event" emits a new update.

How has this change been tested?

How can we measure success?

Have we considered potential risks?

Images

Accessibility

@github-actions

github-actions Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

case object DeleteChannelEvent extends LiveActivityEventType { val asString = "channel-delete" }

object LiveActivityEventType {
val values: Seq[LiveActivityEventType] = Seq(

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not used anywhere

@github-actions

github-actions Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

case object CreateChannelEvent extends LiveActivityEventType { val asString = "channel-create" }
case object StartLiveActivityEvent extends LiveActivityEventType { val asString = "broadcast-start" }
case object UpdateLiveActivityEvent extends LiveActivityEventType { val asString = "broadcast-update" }
case object UpdateStateChangeLiveActivityEvent extends LiveActivityEventType { val asString = "broadcast-update" }

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is needed to be able to dedupe, but the string value should mean it's processed as a standard update event once pushed to eventbus to the live activities service.

@github-actions

Copy link
Copy Markdown
Contributor

@github-actions

Copy link
Copy Markdown
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant