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

refactor PSBT file #2076

Closed
wants to merge 8 commits into from
Closed

refactor PSBT file #2076

wants to merge 8 commits into from

Conversation

jasonandjay
Copy link
Member

The motivation for doing this is to simplify the PSBT code and put together the more closely related logic through the split mode to facilitate subsequent maintenance and splitting.
Mentioned here before #1993
So I did three things

  1. Move payments-related logic from psbtutil to payments, which is more appropriate.
  2. Split the auxiliary functions in class PSBT into global, input and output in the PSBT directory
  3. I would like to hear your opinions. Do you have any intention to dismantle it separately? Like BIP174, I can do this if I have the opportunity.

@junderw junderw requested review from motorina0 and junderw April 12, 2024 09:31
@jasonandjay
Copy link
Member Author

maybe we can ingore this

too many conflicting files!!

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.

1 participant