Skip to content

(Minor) Move payerKey/payerNote to OfferPaymentMetaData class#821

Merged
pm47 merged 1 commit intomasterfrom
offer-metadata-followup
Oct 17, 2025
Merged

(Minor) Move payerKey/payerNote to OfferPaymentMetaData class#821
pm47 merged 1 commit intomasterfrom
offer-metadata-followup

Conversation

@pm47
Copy link
Member

@pm47 pm47 commented Oct 17, 2025

This removes boiler plate code when integrating the library.

This removed boiler plate code when integrating the library.
@pm47 pm47 changed the title (Minor) Move payerKey/payerNote to OfferPaymentData class (Minor) Move payerKey/payerNote to OfferPaymentMetaData class Oct 17, 2025
@pm47 pm47 requested a review from robbiehanson October 17, 2025 13:01
@pm47 pm47 merged commit 57d8c8f into master Oct 17, 2025
2 checks passed
@pm47 pm47 deleted the offer-metadata-followup branch October 17, 2025 13:28
dpad85 added a commit to ACINQ/phoenix that referenced this pull request Oct 17, 2025
See ACINQ/lightning-kmp#821

Also limit the description input to 64 chars for Bolt12 offers
as the description + payer note cannot exceed 64 chars.
dpad85 added a commit to ACINQ/phoenix that referenced this pull request Oct 23, 2025
This commit upgrades lightning-kmp to v1.11.0, which brings taproot
channels to Phoenix. It also updates kotlin and cleans up legacy code.

Notably, the tools for the legacy channel data backup have been
removed, and the import-channel-data helper can be removed.

The spend-from-channel-address screen has been disabled. The
import-channel-data screen is obsolete and has been removed.

* Use abstract payerKey and payerNote in offer metadata

See ACINQ/lightning-kmp#821

Also limit the description input to 64 chars for Bolt12 offers
as the description + payer note cannot exceed 64 chars.

---------

Co-authored-by: Robbie Hanson <304604+robbiehanson@users.noreply.github.com>
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