Skip to content

Conversation

ItoroD
Copy link
Collaborator

@ItoroD ItoroD commented Jul 7, 2025

Description

Allows you to create a Psbt from an unsigned transaction.

Changelog notice

  • Create Psbt from an unsigned transaction.

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

New Features:

  • I've added tests for the new feature
  • I've added docs for the new feature

@ItoroD
Copy link
Collaborator Author

ItoroD commented Jul 7, 2025

I hope we can merge @reez #804 (when its ready for review) before merging this PR. #804 will allow easily add test for this.

@reez
Copy link
Collaborator

reez commented Jul 7, 2025

I hope we can merge @reez #804 (when its ready for review) before merging this PR. #804 will allow easily add test for this.

can do

@ItoroD ItoroD force-pushed the expose-from-unsigned-tx branch 3 times, most recently from 9f59c1b to 45494d3 Compare July 9, 2025 17:47
@ItoroD ItoroD force-pushed the expose-from-unsigned-tx branch from 45494d3 to 5dd6863 Compare July 9, 2025 19:42
@ItoroD
Copy link
Collaborator Author

ItoroD commented Jul 9, 2025

rebased!

///
/// If transactions is not unsigned.
#[uniffi::constructor]
pub(crate) fn from_unsigned_tx(tx: Arc<Transaction>) -> Result<Arc<Psbt>, PsbtError> {
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍 nice, follows existing pattern for return type of similar methods

Copy link
Collaborator

@reez reez left a comment

Choose a reason for hiding this comment

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

ACK 5dd6863

great addition!

@reez reez merged commit cf9dfcc into bitcoindevkit:master Jul 9, 2025
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants