Skip to content

chore: better update ux#1254

Merged
jonathanlab merged 1 commit intomainfrom
fix/ph-code/update-ux-states
Mar 16, 2026
Merged

chore: better update ux#1254
jonathanlab merged 1 commit intomainfrom
fix/ph-code/update-ux-states

Conversation

@tatoalo
Copy link
Contributor

@tatoalo tatoalo commented Mar 16, 2026

Encountered some weird updating UX in ph-code:

  • red X on open when the app's boot check was still in-flight (screenshot)
  • "Check now" doing nothing when an update was already downloaded, clicking "Check now" would silently get stuck. Now shows "Update vX.Y.Z ready to install" with a green checkmark as expected
  • no download feedback, UI showed "Checking for updates..." the whole time. Now transitions to "Downloading update..." (both in the settings panel and the toast) as soon as the download begins (may be slow depending on CDN and user network)
ph_code_old_update

@tatoalo tatoalo self-assigned this Mar 16, 2026
@tatoalo tatoalo marked this pull request as ready for review March 16, 2026 12:40
@tatoalo tatoalo force-pushed the fix/ph-code/update-ux-states branch from 77231f0 to 3adf0a5 Compare March 16, 2026 12:42
@tatoalo tatoalo requested a review from a team March 16, 2026 12:43
@jonathanlab jonathanlab merged commit 8c85de8 into main Mar 16, 2026
14 checks passed
@jonathanlab jonathanlab deleted the fix/ph-code/update-ux-states branch March 16, 2026 12:49
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