Skip to content

Check offline updates for incompatibility#2361

Merged
mcm001 merged 8 commits intoPhotonVision:mainfrom
samfreund:gatekeep-offline-update
Feb 18, 2026
Merged

Check offline updates for incompatibility#2361
mcm001 merged 8 commits intoPhotonVision:mainfrom
samfreund:gatekeep-offline-update

Conversation

@samfreund
Copy link
Member

@samfreund samfreund commented Feb 13, 2026

Description

We've been experiencing a variety of issues where people are performing an offline update with an incompatible jar, which leads to bricking their device. This PR aims to help users avoid this issue. We will deny offline updates built for a different platform, as well asking for confirmation when updating with a dev jar, or a jar built for a different year than the current image.

Note that this functionality is entirely dependent on the filename of the jar, and does not use any metadata.

This PR additionally increases the snackbar message timeout from 2 to 5 seconds, as we've found 2 seconds isn't sufficient to fully read the message.

closes #2357

Meta

Merge checklist:

  • Pull Request title is short, imperative summary of proposed changes
  • The description documents the what and why, including events that led to this PR
  • If this PR changes behavior or adds a feature, user documentation is updated
  • If this PR touches photon-serde, all messages have been regenerated and hashes have not changed unexpectedly
  • If this PR touches configuration, this is backwards compatible with all settings going back to the previous seasons's last release (seasons end after champs ends)
  • If this PR touches pipeline settings or anything related to data exchange, the frontend typing is updated
  • If this PR addresses a bug, a regression test for it is added

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

Labels

backend Things relating to photon-core and photon-server frontend Having to do with PhotonClient and its related items

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants