Skip to content

Allow listing confirmed transactions - #18

Merged
artfuldev merged 6 commits into
masterfrom
allow-listing-transactions
Oct 16, 2021
Merged

Allow listing confirmed transactions#18
artfuldev merged 6 commits into
masterfrom
allow-listing-transactions

Conversation

@artfuldev

@artfuldev artfuldev commented Oct 16, 2021

Copy link
Copy Markdown
Contributor

Resolves #8
Closes #2

We add an API to get confirmed transactions - we filter out unconfirmed transactions. We have a record to pass this information back to the API consumer. The demo application is updated to keep syncing until a confirmed transaction with the broadcasted id is available when listed.

image

@artfuldev

Copy link
Copy Markdown
Contributor Author

Do we want to return all transactions and filter out unconfirmed transactions in the consumer? bdk-cli only lists confirmed transactions so I thought of using that.

@artfuldev
artfuldev merged commit ea24fbb into master Oct 16, 2021
@artfuldev
artfuldev deleted the allow-listing-transactions branch October 16, 2021 22:42
@notmandatory

Copy link
Copy Markdown
Member

For this proof of concept demo I think it's ok to not filter and return what ever the bdk lib returns.

notmandatory pushed a commit to notmandatory/bdk-ffi that referenced this pull request Dec 12, 2021
…ransactions

Allow listing confirmed transactions
@notmandatory notmandatory added this to the 0.2.0 milestone Feb 28, 2022
thunderbiscuit pushed a commit to thunderbiscuit/bdk-ffi that referenced this pull request Mar 1, 2022
…ransactions

Allow listing confirmed transactions
thunderbiscuit added a commit to thunderbiscuit/bdk-ffi that referenced this pull request Oct 26, 2022
Update library to new bdk-ffi API version 0.7.0
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.

Allow verifying a confirmed transaction Demo tracking issue

2 participants