Skip to content

fix: critical update banner for Windows <=5.2.0 - #742

Merged
abose merged 1 commit into
mainfrom
critical-windows-update-banner
Oct 28, 2026
Merged

abose merged 1 commit into
mainfrom
critical-windows-update-banner

Conversation

@abose

@abose abose commented Aug 27, 2026

Copy link
Copy Markdown
Member

What

Adds a CRITICAL_WINDOWS_UPDATE_5_2_0 banner notification to docs/appNotifications/prod, shown only on Windows desktop installs at version <=5.2.0, directing users to phcode.io to download a current build.

Banner

Critical Update Required
This version of Phoenix Code for Windows has a critical stability issue. Please download and install the latest version from phcode.io. [ Download ]

The Download button calls Phoenix.app.openURLInDefaultBrowser('https://phcode.io'), matching the existing lifetime-deal banner.

Config

Field Value
PLATFORM win — Windows desktop only
FOR_VERSIONS <=5.2.0
DANGER_SHOW_ON_EVERY_BOOT true
USER_TYPE not set — all users

No notification_ack class, so clicking Download does not suppress the banner. It reappears every launch until the user actually updates past 5.2.0.

Translations

de, fr, it, ja, ko and pt-BR are hand-written in this PR rather than left to the auto-translation action, so the banner ships fully localized without waiting for a follow-up bot PR. All 7 files validated as parseable JSON.

Deploying

Per docs/appNotifications/readme.md, merging to main is not sufficient — main must then be merged into the update-notifications branch to push this to the installed base.

🤖 Generated with Claude Code

https://claude.ai/code/session_011ifem1U6QZyHLTLs8EVitr

@sonarqubecloud

Copy link
Copy Markdown

Windows builds below 5.3.0 have a critical stability issue. Show a
bottom banner on those installs pointing users to phcode.io to download a
current build. Scoped to PLATFORM "win" only.

Below 5.2.5 the banner shows on every boot and omits notification_ack, so
it keeps reappearing until the user updates. From 5.2.5 up to 5.3.0 the
normal auto-update path works, so the banner is shown just once.

Translations for de, fr, it, ja, ko and pt-BR are hand-written here rather
than waiting on the auto-translation action, since this is urgent.

Claude-Session: https://claude.ai/code/session_011ifem1U6QZyHLTLs8EVitr
Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
@abose
abose force-pushed the critical-windows-update-banner branch from bd5d91f to 44d33e4 Compare October 28, 2026 07:32
@sonarqubecloud

Copy link
Copy Markdown

@abose
abose merged commit edf2510 into main Oct 28, 2026
2 of 6 checks passed
@abose
abose deleted the critical-windows-update-banner branch October 28, 2026 07:33
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