Skip to content

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

Open
abose wants to merge 1 commit into
mainfrom
critical-windows-update-banner
Open

fix: critical update banner for Windows <=5.2.0#742
abose wants to merge 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

Windows builds at or below 5.2.0 have a critical stability issue. Show a
bottom banner on those installs pointing users to phcode.io to download a
current build.

Shows on every boot and omits notification_ack, so it keeps reappearing
until the user actually updates past 5.2.0. Scoped to PLATFORM "win" only.

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.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011ifem1U6QZyHLTLs8EVitr
@sonarqubecloud

Copy link
Copy Markdown

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