Skip to content

Jon/propose fulfilling payment uri#62

Merged
peachbits merged 1 commit intomasterfrom
jon/propose-fulfilling-payment-uri
Dec 4, 2025
Merged

Jon/propose fulfilling payment uri#62
peachbits merged 1 commit intomasterfrom
jon/propose-fulfilling-payment-uri

Conversation

@Jon-edge
Copy link
Copy Markdown
Contributor

@Jon-edge Jon-edge commented Nov 29, 2025

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes
  • No

Dependencies

none

Description

none

@Jon-edge Jon-edge force-pushed the jon/propose-fulfilling-payment-uri branch from 541993d to 8e9e68b Compare November 29, 2025 00:12
src/types.ts Outdated
memo?: string
}

export interface ProposePaymentUriOpts {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Where is this used?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Removed

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nitpick - Not a bad thing to include checkpoint updates but they're not relevant to the PR and can be left for new package publishes

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Removed

@Jon-edge Jon-edge force-pushed the jon/propose-fulfilling-payment-uri branch 2 times, most recently from ee5195d to 59bc86c Compare December 2, 2025 18:42
@peachbits peachbits force-pushed the jon/propose-fulfilling-payment-uri branch from 59bc86c to 91c77d6 Compare December 4, 2025 19:39
@peachbits peachbits merged commit 017d30e into master Dec 4, 2025
3 checks passed
@peachbits peachbits deleted the jon/propose-fulfilling-payment-uri branch December 4, 2025 19:44
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