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

Expose spend module and coins fields #865

Merged
merged 2 commits into from
Dec 11, 2023

Conversation

edouardparis
Copy link
Member

In order to externally use create_spend, the spend module must be exposed and the listcoinsentry must include the derivation_index and is_change fields.

Copy link
Member

@darosior darosior left a comment

Choose a reason for hiding this comment

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

Makes sense to expose those fields. Need to document them in doc/API.md though.

src/commands/mod.rs Show resolved Hide resolved
src/commands/mod.rs Outdated Show resolved Hide resolved
Copy link
Member

@darosior darosior left a comment

Choose a reason for hiding this comment

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

utACK 572567a

doc/API.md Show resolved Hide resolved
@darosior darosior merged commit 23e834a into wizardsardine:master Dec 11, 2023
18 checks passed
@edouardparis edouardparis deleted the expose-spend-module branch December 11, 2023 16:18
darosior added a commit that referenced this pull request Dec 12, 2023
…oin selection

a732468 spend: check max feerate (edouardparis)
4ccecd1 Use create_spend to calculate amount left to select (edouardparis)

Pull request description:

  Fixes #822. based on #865

ACKs for top commit:
  jp1ac4:
    ACK a732468.

Tree-SHA512: aa2728218d9ccf7b6511f881d88b262522705b4162b5b4235560626ee8b5b8c6d57e743ebfaf85dc633d6e3d2380badfe354a6dc4d19b89d3f380c63e123704e
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