Skip to content

feat: Add from_unsigned_tx to Psbt #802

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

Merged
merged 1 commit into from
Jul 9, 2025

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