Skip to content

Conversation

@ovitrif
Copy link
Collaborator

@ovitrif ovitrif commented Jul 23, 2025

Related to #202

FIGMA: ConfirmScreen | ConnectingScreen

This implements lnurl channel functionality.

Description

  • Lnurl channel QR code scan handling
  • Lnurl channel confirm screen

Preview

Via Scan Via LN Manual Setup
lnurlChannel.mp4
lnurlChannelQr.mp4

QA Notes

LNURL Channel

  • in Env.kt, change ElectrumServers.REGTEST to
    host = "__YOUR_NETWORK_IP__",
    tcp = 60001,
    
  • uninstall old app and install fresh one
  • checkout bitkit-docker repo
  • setup docker-compose.yml
    • change DOMAIN to http://__YOUR_NETWORK_IP__:3000
    • set lnd's --externalip=127.0.0.1 to --externalip=__YOUR_NETWORK_IP__
  • reset docker data
    • docker compose down --volumes
    • rm -rf ./lnd ./lnurl-server/data
  • docker compose up --build
  • mine blocks: ./bitcoin-cli mine 101
  • fund LND wallet:
    • get address: curl -s http://localhost:3000/address | jq -r .address
    • fund LND wallet: ./bitcoin-cli send 0.2
    • mine block ./bitcoin-cli mine 1
  • generate LNURL channel: http://localhost:3000/channel
  • paste lnurl into app and complete the flow
  • mine blocks: ./bitcoin-cli mine 6

Tests

  • test lnurl channel when peer is already connected
  • test lnurl channel when peer is not connected
  • test lnurl channel QR via manual setup

@ovitrif ovitrif self-assigned this Jul 23, 2025
@ovitrif ovitrif marked this pull request as ready for review July 23, 2025 21:12
@ovitrif ovitrif requested a review from jvsena42 July 23, 2025 21:12
@ovitrif ovitrif force-pushed the feat/lnurl-channel branch from 241151c to d0bc9c2 Compare July 24, 2025 08:54
@ovitrif ovitrif requested a review from jvsena42 July 24, 2025 12:10
@ovitrif ovitrif merged commit cc05055 into master Jul 24, 2025
2 checks passed
@ovitrif ovitrif deleted the feat/lnurl-channel branch July 24, 2025 13:49
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.

3 participants