-
Notifications
You must be signed in to change notification settings - Fork 10
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
SDK update #70
SDK update #70
Conversation
8ec8c4a
to
430b403
Compare
a29d7c5
to
ecac71c
Compare
f96cdef
to
c1eaaeb
Compare
38b710d
to
1969aac
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.
LGTM, thanks for doing this.
BTW 2 tests fail locally when using a
BLOCKFROST_IPFS_PROJECT_ID
with NETWORK: IPFS
and a BLOCKFROST_PROJECT_ID
with NETWORK: Cardano mainnet
. 🤔
go test ./...
? github.com/blockfrost/blockfrost-go/example/accounts [no test files]
? github.com/blockfrost/blockfrost-go/example/addresses [no test files]
? github.com/blockfrost/blockfrost-go/example/basic [no test files]
? github.com/blockfrost/blockfrost-go/example/genesis [no test files]
? github.com/blockfrost/blockfrost-go/example/ipfs [no test files]
? github.com/blockfrost/blockfrost-go/example/metadata [no test files]
? github.com/blockfrost/blockfrost-go/example/nutlink [no test files]
? github.com/blockfrost/blockfrost-go/example/pools [no test files]
? github.com/blockfrost/blockfrost-go/example/scripts [no test files]
? github.com/blockfrost/blockfrost-go/example/scripts/scripts_dump [no test files]
? github.com/blockfrost/blockfrost-go/internal/version [no test files]
WARNING: Cannot parse part of the signature header, key "v42" is not supported by this version of Blockfrost SDK.
--- FAIL: TestResourceMetricsIntegration (0.06s)
api_metrics_test.go:22: got null []
--- FAIL: TestResourceMetricsEndpointsIntegration (0.09s)
api_metrics_test.go:38: got null []
FAIL
FAIL github.com/blockfrost/blockfrost-go 8.499s
FAIL
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.
LGTM 🎉
go test ./...
ok github.com/blockfrost/blockfrost-go 13.499s
close #68
Added
Mempool
,MempoolAll
,MempoolTx
,MempoolByAddress
,MempoolByAddressAll
TransactionEvaluate
andTransactionEvaluateUTXOs
AddressUTXOsAsset
andAddressUTXOsAssetAll
(orig PR Add AddressUTXOsAsset and AddressUTXOsAssetAll functions #65)VerifyWebhookSignature
for WebhooksBlocksAddresses
for list of addresses affected by a specific blockFixed
nil
instead of setting these fields to their default value based on data type (eg. "" for string fields that actuallynull
in JSON response)Changed