Skip to content

wip: Sketch out hybrid QR transport support#34

Merged
iinuwa merged 6 commits into
mainfrom
hybrid
Jun 3, 2025
Merged

wip: Sketch out hybrid QR transport support#34
iinuwa merged 6 commits into
mainfrom
hybrid

Conversation

@iinuwa
Copy link
Copy Markdown
Member

@iinuwa iinuwa commented May 25, 2025

  • add GTK changes and QR code lib for UI flow
  • Add handler trait and stream to pull into view model
  • Add dummy hybrid QR handler for demonstrating flow

@AlfioEmanueleFresta
Copy link
Copy Markdown
Member

LGTM so far (I tried running this)

@iinuwa iinuwa force-pushed the hybrid branch 3 times, most recently from 1b62327 to b359bf4 Compare June 3, 2025 03:34
@iinuwa
Copy link
Copy Markdown
Member Author

iinuwa commented Jun 3, 2025

This still needs a couple of things:

  • UI updates after receiving BLE advert and after tunnel is established (needs support from libwebauthn)
  • remove some Dummy implementations

There's a weird race condition or something: the UI renders correctly if I add breakpoints, but misses the initial update. I'm wondering if this is some sort of problem with mixing async runtimes. To avoid a big rebase, I'm going to go ahead and merge this in, switch over to Tokio every, and then come back to this.

@iinuwa iinuwa marked this pull request as ready for review June 3, 2025 04:05
@iinuwa iinuwa merged commit ce11c10 into main Jun 3, 2025
1 check passed
@iinuwa iinuwa deleted the hybrid branch June 3, 2025 04:06
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