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

LNURL: Use Lightning description template in LNURL metadata #3667

Merged
merged 3 commits into from
Apr 24, 2022

Conversation

dennisreimann
Copy link
Member

@dennisreimann dennisreimann commented Apr 21, 2022

Unifies the invoice description for the Lightning and LNURL payment methods and fixes #3634.

#3655 brought up a similar topic. In efc1a0d I unified the LNURL meta data with the Lightning invoice meta data, so that individual POS item purchases can be tracked using the invoice meta data.

de47d3a adds an order ID for individual POS item purchases. This is so that it is consistent with the Crowdfund order IDs and it is also used in the previously mentioned commit.

Unifies the invoice description for the Lightning and LNURL payment methods and fixes btcpayserver#3634.
For consistency with the Lightning payment method. Closes btcpayserver#3655.
@pavlenex pavlenex added this to the 1.6.0 milestone Apr 23, 2022
@NicolasDorier NicolasDorier merged commit 54e106e into btcpayserver:master Apr 24, 2022
@dennisreimann dennisreimann deleted the lnurl-metadata branch April 25, 2022 06:28
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.

LNURL does not pass description to LN invoice
3 participants