Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix lnd.py #126

Merged
merged 1 commit into from
Dec 5, 2022
Merged

Fix lnd.py #126

merged 1 commit into from
Dec 5, 2022

Conversation

nicheosala
Copy link
Contributor

SatSate was not able to create an invoice through LND, because of a typo. Now this problem has been solved.

However, despite being able to create invoices through LND, the LND support is still broken. In fact, if you try to send 1 sat from the web interface, SatSale will create an invoice and, after a few seconds, it will mark the invoce as paid and close the web page. All without being able to pay the invoice.

SatSate was not able to create an invoice through LND, because of a typo.

Anyway, despite being able to create invoices through LND, the LND support is still broken. In fact, if you try to send 1 sat from the web interface, SatSale will create an invoice and, after very few seconds, it will mark the invoce as paid and close the web page. All without being able to pay the invoice for real!
@nickfarrow nickfarrow merged commit 1045d7c into SatSale:master Dec 5, 2022
@nickfarrow
Copy link
Member

Thanks!!

@nickfarrow
Copy link
Member

Is the 1 sat issue the same as:
#92

with tmp fix:
#96

@kristapsk
Copy link
Member

Yes, the second issue mentioned should be same as #92.

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.

None yet

3 participants