Skip to content

Use refresh event for WebApp updates [WPB-22420]#21838

Open
screendriver wants to merge 1 commit into
mainfrom
feature-flags
Open

Use refresh event for WebApp updates [WPB-22420]#21838
screendriver wants to merge 1 commit into
mainfrom
feature-flags

Conversation

@screendriver

@screendriver screendriver commented Jul 14, 2026

Copy link
Copy Markdown
Member
TaskWPB-22420 [Web] General maintenance ticket for PR merges

Context

When the “Update now” banner is used inside Wire Desktop, the WebApp currently publishes WebAppEvents.LIFECYCLE.RESTART.

Wire Desktop interprets this as a request to relaunch the Electron application. During that relaunch, the WebApp URL is reconstructed from the configured environment, so query parameters such as enabled-features are lost.

A WebApp update only requires the existing WebViews to be refreshed. It does not require the Electron application itself to restart.

Changes

This changes App.refresh() to publish WebAppEvents.LIFECYCLE.REFRESH in desktop environments.

The browser behavior remains unchanged and still reloads the current page directly.

Existing uses of WebAppEvents.LIFECYCLE.RESTART are intentionally unchanged because they represent operations that may require a real desktop application restart.

Result

Wire Desktop can now distinguish between refreshing the WebApp and restarting the Electron application.

This allows the desktop client to reload the existing WebView while preserving its current URL, including startup feature-toggle query parameters.

See also wireapp/wire-desktop#9696

Use the lifecycle refresh event when the WebApp update banner reloads inside Wire Desktop, preserving the existing browser reload behavior.
@sonarqubecloud

Copy link
Copy Markdown

zskhan
zskhan previously approved these changes Jul 14, 2026
@screendriver
screendriver changed the base branch from dev to main July 14, 2026 11:40
@screendriver
screendriver dismissed zskhan’s stale review July 14, 2026 11:40

The base branch was changed.

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.

2 participants