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

feat: add optional CBOR representation to GET_ACCOUNT_INFO #271

Merged
merged 2 commits into from
Dec 5, 2024

Conversation

iccicci
Copy link
Collaborator

@iccicci iccicci commented Nov 20, 2024

This solves #253

@iccicci iccicci force-pushed the feat/GET_ACCOUNT_INFO_cbor branch from 5c26d93 to b536bf6 Compare November 20, 2024 08:37
@iccicci iccicci force-pushed the feat/GET_TRANSACTION_cbor branch from 09b3697 to 36edc11 Compare November 25, 2024 10:36
@iccicci iccicci force-pushed the feat/GET_ACCOUNT_INFO_cbor branch from b536bf6 to a12326e Compare November 25, 2024 10:37
@iccicci iccicci force-pushed the feat/GET_TRANSACTION_cbor branch from 36edc11 to 7eee599 Compare November 29, 2024 10:15
@iccicci iccicci force-pushed the feat/GET_ACCOUNT_INFO_cbor branch from a12326e to d82aab9 Compare November 29, 2024 10:16
@iccicci iccicci force-pushed the feat/GET_TRANSACTION_cbor branch from 7eee599 to 36b7266 Compare November 29, 2024 12:06
@iccicci iccicci force-pushed the feat/GET_ACCOUNT_INFO_cbor branch from d82aab9 to 3714972 Compare November 29, 2024 12:07
@iccicci iccicci force-pushed the feat/GET_TRANSACTION_cbor branch from 36b7266 to 393db04 Compare November 29, 2024 12:26
@iccicci iccicci force-pushed the feat/GET_ACCOUNT_INFO_cbor branch from 3714972 to ae7b387 Compare November 29, 2024 12:26
@iccicci iccicci force-pushed the feat/GET_TRANSACTION_cbor branch from 393db04 to 99ba285 Compare November 29, 2024 12:39
@iccicci iccicci force-pushed the feat/GET_ACCOUNT_INFO_cbor branch from ae7b387 to f4e2a97 Compare November 29, 2024 12:39
@iccicci iccicci force-pushed the feat/GET_TRANSACTION_cbor branch from 99ba285 to 671636e Compare November 29, 2024 12:54
@iccicci iccicci force-pushed the feat/GET_ACCOUNT_INFO_cbor branch from f4e2a97 to c07d859 Compare November 29, 2024 12:54
@iccicci iccicci force-pushed the feat/GET_TRANSACTION_cbor branch from 7d3b2b9 to 5f3ab98 Compare December 2, 2024 08:28
@iccicci iccicci force-pushed the feat/GET_ACCOUNT_INFO_cbor branch from c07d859 to 7ea6882 Compare December 2, 2024 13:33
@iccicci iccicci force-pushed the feat/GET_ACCOUNT_INFO_cbor branch from 7ea6882 to 86ebd8e Compare December 3, 2024 10:24
@iccicci iccicci changed the base branch from feat/GET_TRANSACTION_cbor to master December 3, 2024 10:24
@iccicci iccicci marked this pull request as ready for review December 3, 2024 10:24
Copy link
Contributor

@slowbackspace slowbackspace left a comment

Choose a reason for hiding this comment

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

lgtm! please add unit test for get-account-info to https://github.com/blockfrost/blockfrost-websocket-link/blob/ba7a3176bcb76639abf0c5103f91d04a5f2ec65e/test/unit/tests/methods/get-account-info.test.ts to cover new param.

Check out comments in the file on how to automatically record needed requests using nock

@slowbackspace slowbackspace force-pushed the feat/GET_ACCOUNT_INFO_cbor branch from 38f10c0 to 4ebfd1e Compare December 5, 2024 10:38
@slowbackspace slowbackspace merged commit 212fe42 into master Dec 5, 2024
3 checks passed
@iccicci iccicci deleted the feat/GET_ACCOUNT_INFO_cbor branch December 5, 2024 14:32
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