-
Notifications
You must be signed in to change notification settings - Fork 36
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
Update tx_info request to include inputs #829
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.
I don't understand. Put more words in the commit or something
CHANGELOG.md
Outdated
@@ -5,6 +5,7 @@ | |||
- added: (Ethereum/Solana) Support additional tokens provided by info server | |||
- added: (Thorchain) Add Midgard API for transaction queries | |||
- changed: (Thorchain) Move most Thorchain-specific functionality out of CosmosEngine and into to ThorchainEngine subclass | |||
- fixed: Update tx_info request to include inputs |
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.
Add (ADA)
to this changelog so it's not so ambiguous
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.
Why not fixed: (ADA) Update tx_info request to include inputs in order to fix all transactions show as receive bug
a65da87
to
148765b
Compare
CHANGELOG.md
Outdated
@@ -5,6 +5,7 @@ | |||
- added: (Ethereum/Solana) Support additional tokens provided by info server | |||
- added: (Thorchain) Add Midgard API for transaction queries | |||
- changed: (Thorchain) Move most Thorchain-specific functionality out of CosmosEngine and into to ThorchainEngine subclass | |||
- fixed: Update tx_info request to include inputs |
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.
Why not fixed: (ADA) Update tx_info request to include inputs in order to fix all transactions show as receive bug
148765b
to
f1031f1
Compare
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.
Approved this. Please change the changelog entry though
The Koios tx_info endpoint changed and you must now request input data
f1031f1
to
80ce08c
Compare
CHANGELOG
Does this branch warrant an entry to the CHANGELOG?
Dependencies
noneDescription
none