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

Update tx_info request to include inputs #829

Merged
merged 1 commit into from
Sep 5, 2024
Merged

Conversation

peachbits
Copy link
Contributor

@peachbits peachbits commented Sep 5, 2024

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes
  • No

Dependencies

none

Description

none

Copy link
Collaborator

@samholmes samholmes left a 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
Copy link
Collaborator

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

Copy link
Collaborator

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

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
Copy link
Collaborator

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

Copy link
Collaborator

@samholmes samholmes left a 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
@peachbits peachbits merged commit d628f5b into master Sep 5, 2024
2 checks passed
@peachbits peachbits deleted the matthew/cardano-inputs branch September 5, 2024 20:31
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