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

PR for Supporting Get ETH Tx Receipt by Tx Hash #62

Closed
wants to merge 6 commits into from

Conversation

avislash
Copy link
Contributor

#61

@avislash avislash force-pushed the add-get-txnreceipt-by-hash branch from c9d711f to db1de1b Compare October 15, 2021 10:30
@@ -1,3 +1,8 @@
module github.com/nanmu42/etherscan-api

go 1.13

require (
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These only got added to make testing in proxy_e2e_test.go easier. I can simplify the test case by using a simpler txn receipt and test more similar to the other cases just let me know.

@avislash avislash force-pushed the add-get-txnreceipt-by-hash branch from b00a9dd to 90530de Compare October 15, 2021 10:50
@avislash
Copy link
Contributor Author

CI fails because it can't access the ETHERSCAN_API_KEY secret. Not sure how to fix this, I have it set in my repo to a test key but for some reason it doesn't seem to be working. Any ideas on how to fix?

@avislash avislash force-pushed the add-get-txnreceipt-by-hash branch from 90530de to 6eb9c6e Compare October 15, 2021 17:48
Also:
  *Added Test for GetTxReceipt

  *Updated CI to download go mod dependencies
@nanmu42
Copy link
Owner

nanmu42 commented Oct 18, 2021

Closed via #61 .

@nanmu42 nanmu42 closed this Oct 18, 2021
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