Skip to content

Conversation

@pwltr
Copy link
Contributor

@pwltr pwltr commented Jan 23, 2026

Description

  • show a “Syncing Node” overlay when opening Send if the node is not running yet
  • defer validation checks until after sync

Linked Issues/Tasks

Closes #339

Screenshot / Video

Unified invoice:

Simulator.Screen.Recording.-.iPhone.17.-.2026-01-16.at.01.00.09.mov

QuickPay (deeplink):

Simulator.Screen.Recording.-.iPhone.17.-.2026-01-16.at.01.00.09.mov

QA Notes

Use a deeplink like

xcrun simctl openurl "iPhone 17" "bitcoin:bcrt1qmd722klk04yph86ky8jz9gvj6g8n9kjep9zj7d?amount=0.0000002&lightning=LNBCRT200N1P5H8H4HDQQNP4QDRM0Y4AT84E48QVDN8CSWVE204SF6FFRR3W3AK904GE3JSEGLP4GPP5VCSLQ6RPA2RHQJ40LG6ELHE4JA0548TNQZTX4UFSTKAJ8FAHKLASSP5E2EQTTN20PF2HV7EZQCDKY43DSCVLXXR5LC6E920GDR3GXJ0QYSQ9QYYSGQCQPCXQRRSSRZJQ29GJY9SQJRRP48TZ7HJ2E5VM4L2DUKC4CSF2MN6QM32U3HTED5LEAPYQQQQQQQF95QQQQLGQQQQQQGQ2Q9T08YP2026U697073LS2FRT0EY9MKEYQASUTM5DV9KV5FZS5X2UPPHLFYU4C3Q34LJ8GRTLQ2YG0SWPZJMSK74MV6WHWUEZWG870AHCQGD3XDW"

@pwltr pwltr marked this pull request as ready for review January 26, 2026 09:18
@pwltr pwltr force-pushed the feat/node-connecting-ui branch from 08d95b0 to 9555fd1 Compare January 26, 2026 09:21
@claude

This comment has been minimized.

Comment on lines +217 to +221
// DEBUG: Add artificial delay to test sync overlay UI
#if DEBUG
try? await Task.sleep(nanoseconds: 10_000_000_000) // 5 seconds
#endif

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we still want to leave this?

@claude
Copy link

claude bot commented Jan 26, 2026

The comment on line 219 of Bitkit/Services/LightningService.swift states "5 seconds" but the actual delay is 10 seconds (10_000_000_000 nanoseconds = 10 seconds).

While this is DEBUG-only code for testing the sync overlay UI, the comment should match the implementation to avoid confusion.

Suggested fix: Update the comment to say "10 seconds" or change the value to 5_000_000_000 to match the comment.

Reference:

// DEBUG: Add artificial delay to test sync overlay UI
#if DEBUG
try? await Task.sleep(nanoseconds: 10_000_000_000) // 5 seconds
#endif

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.

[BUG] "Lightning not running" when paying quickly

5 participants