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

Add test for raw transaction flow #17

Merged
merged 3 commits into from
Apr 22, 2024
Merged

Add test for raw transaction flow #17

merged 3 commits into from
Apr 22, 2024

Conversation

JBetz
Copy link
Contributor

@JBetz JBetz commented Apr 19, 2024

Also fixes an inconsistency in fundrawtransaction where the fee asset would only default to the asset being sent in the transaction when an option parameter was provided.

@JBetz JBetz requested a review from Mixa84 as a code owner April 19, 2024 20:05
@JBetz JBetz merged commit d3911fd into dev Apr 22, 2024
1 check passed
@JBetz JBetz deleted the fundrawtransaction-test branch April 22, 2024 11:37
JBetz added a commit that referenced this pull request Apr 23, 2024
* Add test for raw* transaction flow

* When any asset fees is enabled, use transaction asset for fee asset even when options parameter is not provided
JBetz added a commit that referenced this pull request Apr 24, 2024
* Add test for raw* transaction flow

* When any asset fees is enabled, use transaction asset for fee asset even when options parameter is not provided
JBetz added a commit that referenced this pull request May 3, 2024
* Add test for raw* transaction flow

* When any asset fees is enabled, use transaction asset for fee asset even when options parameter is not provided
Mixa84 pushed a commit that referenced this pull request May 6, 2024
* Add test for raw* transaction flow

* When any asset fees is enabled, use transaction asset for fee asset even when options parameter is not provided
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.

2 participants