Skip to content

Add add_data function for TxBuilder op_return #163

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
Jun 23, 2022

Conversation

logosstone
Copy link
Contributor

@logosstone logosstone commented Jun 10, 2022

fixes #132

@logosstone logosstone force-pushed the add_data_implement branch 2 times, most recently from f9ce873 to 410eb82 Compare June 16, 2022 17:52
@logosstone logosstone changed the title add_data fxn for txbuilder op_return Add add_data function for TxBuilder op_return Jun 16, 2022
@logosstone logosstone marked this pull request as ready for review June 16, 2022 17:58
@notmandatory notmandatory self-requested a review June 16, 2022 18:00
@thunderbiscuit
Copy link
Member

thunderbiscuit commented Jun 16, 2022

I tested this and broadcast a transaction out with a message 🔥 🔥 🔥.

The Kotlin type on the other side of this is List<UByte> so it takes a bit of manipulating but it worked on the first try.

@thunderbiscuit thunderbiscuit added this to the Release 0.8.0 milestone Jun 17, 2022
Copy link
Member

@thunderbiscuit thunderbiscuit left a comment

Choose a reason for hiding this comment

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

Very small nit, otherwise I think this is ready for merge.

Tested ACK 410eb82.

@logosstone logosstone force-pushed the add_data_implement branch from 410eb82 to 57ec72e Compare June 22, 2022 21:54
@logosstone logosstone force-pushed the add_data_implement branch from 57ec72e to 548217e Compare June 22, 2022 22:00
Copy link
Member

@notmandatory notmandatory left a comment

Choose a reason for hiding this comment

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

Looks good! ACK 548217e

@thunderbiscuit thunderbiscuit self-requested a review June 23, 2022 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Add TxBuilder addData method
3 participants