Skip to content

feature(update): Version 1.0.1, Smoother progress, better cancellation flow#322

Open
theypsilon wants to merge 4 commits into
ZaparooProject:mainfrom
theypsilon:main
Open

feature(update): Version 1.0.1, Smoother progress, better cancellation flow#322
theypsilon wants to merge 4 commits into
ZaparooProject:mainfrom
theypsilon:main

Conversation

@theypsilon

@theypsilon theypsilon commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Harden update cancellation and reboot flow
    • Keep cancelled updates in a locked stopping state until downloader exits or gets killed after 30 secs.
    • Reboot commands launchs 4 seconds before the count down ends, to compensate for the time the command takes to execute.
  • Zaparoo Update version shown in the UI of the Update screen.
  • Refined update progress details.
  • Refined updater progress display, and status pacing.
    • % now shows decimals when the update is big.
  • Smoother progress bar.

Motivation

Screenshots / recordings

Test plan

Go to the Update option from the main screen. To test the cancel flow, try to interrupt the update.

Checklist

  • just lint is green (zero warnings)
  • just test passes
  • If this touches QML, the FPS counter stays green (≥ 55) at 720p+ and ≥ 30 at 240p
  • If this could affect the MiSTer build, I considered ARM32 implications (see docs/architecture.md)
  • If this adds user-visible strings, they are wrapped in qsTr() (QML) or tr() (C++)
  • I have signed the CLA (first-time contributors only)

Summary by CodeRabbit

  • Chores
    • Updated an optional app dependency to a newer version.

@coderabbitai

coderabbitai Bot commented Jul 2, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

This change updates the optional zaparoo-update dependency version in rust/frontend/Cargo.toml from 0.1.10 to 1.0.1. No other files or exported entities are modified.

Changes

Dependency Update

Layer / File(s) Summary
Update zaparoo-update dependency
rust/frontend/Cargo.toml
Bumped the optional zaparoo-update crate dependency version from 0.1.10 to 1.0.1.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The template is mostly filled, but the visual-change screenshots section is empty and the checklist is incomplete. Add screenshots/recordings for the UI change and either complete or explicitly note the remaining checklist items; fill motivation only if applicable.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title matches the version bump and smoother update/cancellation changes in the PR.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@theypsilon

Copy link
Copy Markdown
Collaborator Author

Mmm, the text from the PR disappeared, Adding it back.

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