Skip to content

Conversation

TheBlueMatt
Copy link
Collaborator

No description provided.

@ldk-reviews-bot
Copy link

ldk-reviews-bot commented Apr 30, 2025

👋 I see @jkczyz was un-assigned.
If you'd like another reviewer assignemnt, please click here.

@ldk-reviews-bot ldk-reviews-bot requested a review from jkczyz April 30, 2025 13:16
@ldk-reviews-bot
Copy link

👋 The first review has been submitted!

Do you think this PR is ready for a second reviewer? If so, click here to assign a second reviewer.

@ldk-reviews-bot ldk-reviews-bot requested a review from jkczyz April 30, 2025 13:33
@ldk-reviews-bot
Copy link

✅ Added second reviewer: @jkczyz

@ldk-reviews-bot
Copy link

✅ Added second reviewer: @wpaulino

@TheBlueMatt TheBlueMatt removed the request for review from jkczyz April 30, 2025 13:58
Copy link
Contributor

@joostjager joostjager left a comment

Choose a reason for hiding this comment

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

Verified that all cherry-picks are in the changelog.

CHANGELOG.md Outdated
# 0.1.3 - Apr 30, 2025 - "Routing Unicode in 2025"

## Bug Fixes
* `Event::InvoiceRecieved` is now only generated once for each `Bolt12Invoice`
Copy link
Contributor

Choose a reason for hiding this comment

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

PR desc says it is specifically when manually_handle_bolt12_invoice is enabled. But statement as it is true too then.

Copy link
Contributor

Choose a reason for hiding this comment

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

Recieved -> Received

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sure, InvoiceReceived is only generated if manually_handle_bolt12_invoice is set :).

Copy link
Contributor

Choose a reason for hiding this comment

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

Trying to make the best of a changelog review...

@TheBlueMatt TheBlueMatt force-pushed the 2025-04-0.1.3-relnotes branch from ccd9444 to 0edff53 Compare April 30, 2025 15:49
@TheBlueMatt
Copy link
Collaborator Author

Fixed the typo Joost caught :(

@TheBlueMatt
Copy link
Collaborator Author

$ git diff-tree -U1 ccd94445c 0edff530e
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 0d7c9d5d6..80652dcb7 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -3,3 +3,3 @@
 ## Bug Fixes
- * `Event::InvoiceRecieved` is now only generated once for each `Bolt12Invoice`
+ * `Event::InvoiceReceived` is now only generated once for each `Bolt12Invoice`
    received matching a pending outbound payment. Previously it would be provided

Copy link
Contributor

@tnull tnull left a comment

Choose a reason for hiding this comment

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

LGTM

@TheBlueMatt TheBlueMatt merged commit b4d5fe2 into lightningdevkit:0.1 Apr 30, 2025
21 of 27 checks passed
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.

7 participants