Skip to content

Comments

Attempt connection multiple times, retrieve latest services before starting notifications#76

Merged
microbit-grace merged 6 commits intoappsfrom
post-flash-connect
Jan 21, 2026
Merged

Attempt connection multiple times, retrieve latest services before starting notifications#76
microbit-grace merged 6 commits intoappsfrom
post-flash-connect

Conversation

@microbit-grace
Copy link
Contributor

@microbit-grace microbit-grace commented Jan 20, 2026

  • Attempt to connect four times before throwing error.
  • Throw immediately if disconnect occurs whilst connecting instead of waiting for timeout.
  • Ensure latest services are found.

- Attempt to connect twice before throwing error.
- Throw immediately if disconnect occurs whilst connecting instead of waiting for timeout.
@microbit-grace microbit-grace marked this pull request as draft January 20, 2026 18:08
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 21, 2026

Deploying microbit-connection with  Cloudflare Pages  Cloudflare Pages

Latest commit: 86723a6
Status: ✅  Deploy successful!
Preview URL: https://5e4006ea.microbit-connection.pages.dev
Branch Preview URL: https://post-flash-connect.microbit-connection.pages.dev

View logs

This fixes the issue where no services are found after bonding and connecting with a micro:bit.
@microbit-grace microbit-grace changed the title Handle Android connection fail after flashing Handle Android connection fail after partial flashing Jan 21, 2026
@microbit-grace microbit-grace marked this pull request as ready for review January 21, 2026 13:57
@microbit-grace microbit-grace changed the title Handle Android connection fail after partial flashing Attempt connection multiple times, retrieve latest services before starting notifications Jan 21, 2026
@microbit-grace microbit-grace merged commit 65549d9 into apps Jan 21, 2026
4 checks passed
@microbit-grace microbit-grace deleted the post-flash-connect branch January 21, 2026 15:03
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