Skip to content

Show a message when no player is online, and a spinner for the update check - #36

Merged
strikeout merged 1 commit into
mainfrom
austin/offline-master-ota-spinner
Oct 26, 2026
Merged

strikeout merged 1 commit into
mainfrom
austin/offline-master-ota-spinner

Conversation

@strikeout

@strikeout strikeout commented Oct 26, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • When the page has no player and no Ableton Link source (no_devices), the Master panel shows a message. It does not show the empty BPM, beat and phrase fields.
  • When you click "Check for update", the page shows a spinner and the text "checking…". The box sends no status during the check. Thus the spinner stops at the first status 1.5 s after the click, or at an error reply.
  • The version changes to 1.22.2 for the host and the firmware.

Test

  • cargo test --locked passes. cargo build --release --locked in esp32/ passes.
  • The host with Ableton Link off and no player shows the message in the Master panel.
  • A browser test on the host page gives the page false OTA states. The spinner shows after the click and stays through an old status. It stops at the status after the check and at an error reply.
  • The box has firmware 1.22.2 from USB. It serves the new page, and the virtual CDJ is device 7.

… check

When the page has no player and no Ableton Link source (no_devices),
the Master panel shows a message. It does not show the empty BPM, beat
and phrase fields.

When you click "Check for update", the page shows a spinner and the text
"checking…". The box sends no status during the check. Thus the page
stops the spinner at the first status that comes 1.5 s after the click,
or at an error reply.
@strikeout
strikeout merged commit e0bf32b into main Oct 26, 2026
7 checks passed
@strikeout
strikeout deleted the austin/offline-master-ota-spinner branch October 26, 2026 23:46
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