Skip to content

Verify the brush, forget on disconnect, report-toast tweaks, new footer - #5

Merged
mannes merged 2 commits into
mainfrom
forget-and-verify
Sep 24, 2026
Merged

mannes merged 2 commits into
mainfrom
forget-and-verify

Conversation

@mannes

@mannes mannes commented Sep 24, 2026

Copy link
Copy Markdown
Contributor
  • Verify the brush: after connecting, read DEVICE_TYPE from the Oral-B service; anything else (a multimeter picked in the chooser) is rejected and forgotten.
  • Auto-reconnect only targets the remembered, verified brush — no more devices[0] fallback.
  • Disconnect = forget (device.forget()), so it isn't silently reconnected. Pause fallback kept for Chrome < 101.
  • Device-report ask once per install instead of once per model; trouble toast on any error, incl. sync errors while connected.
  • Footer: Part of libreble · GitHub · Report an issue · version. Version from the tag in CI, git describe locally, APP_VERSION / Docker build-arg otherwise.

Needs hardware confirm: iO still connects + syncs; picking a non-brush shows the error and never reconnects; after Disconnect a reload doesn't reconnect.

After connecting, read DEVICE_TYPE from the Oral-B service. A device that
doesn't answer it (e.g. a multimeter picked in the all-devices chooser) is
rejected and forgotten. Auto-reconnect now targets only the remembered brush
instead of falling back to any device the origin was ever granted, and
Disconnect revokes the permission via device.forget().
The 'how is it working' ask shows once per install (legacy per-model keys
still count). The trouble toast shows on any error, including sync failures
while connected. The footer links libreble, GitHub and 'Report an issue', and
shows the build version (tag, git describe, or APP_VERSION).
@mannes
mannes merged commit 6ed3c95 into main Sep 24, 2026
2 checks passed
@mannes
mannes deleted the forget-and-verify branch September 24, 2026 07:45
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