-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[Add
] Transaction Builder
#3477
[Add
] Transaction Builder
#3477
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good idea, @cschuchardt88 .
Maybe it would be good to create an issue to present the Neo.Builders with more details and where are the planned use cases.
@shargon @roman-khimov @vncoelho @Jim8y |
If anyone wants to test it, and give me some feedback on anything that needs to be improved. |
@superboyiii @Hecate2 do you have any idea? |
@Jim8y @cschuchardt88 |
You want to test it @Hecate2? Then give some feedback on the builder. |
I think we can try to fill most properties of a tx automatically, unless the user specifies something else, or we do not have access to current chain enviromment. If we leave everything to be filled by the user, the complexity of building a tx is not reduced at all. |
I could create a new |
…ardt88/neo into add/builders/transaction
Description
Easy simple builders for creating with signing, along with creating scripts for transactions.
Change Log
Transaction
,Witness
,Signer
,TransactionAttribute
,WitnessRule
andWitnessCondition
buildersExample
Applies To
Type of change
How Has This Been Tested?
Wrote unit tests
Checklist: