Skip to content
This repository was archived by the owner on Jul 29, 2024. It is now read-only.

Zapper - Replace v1 with v2 endpoints#2

Open
nemtajo wants to merge 4 commits intoJKorf:mainfrom
nemtajo:main
Open

Zapper - Replace v1 with v2 endpoints#2
nemtajo wants to merge 4 commits intoJKorf:mainfrom
nemtajo:main

Conversation

@nemtajo
Copy link
Copy Markdown

@nemtajo nemtajo commented Jan 11, 2023

Changes made to reflect article published by Zapper:
https://blog.zapper.fi/updates-to-zappers-public-apis-4933a2967f57

@nemtajo
Copy link
Copy Markdown
Author

nemtajo commented Feb 6, 2023

@JKorf please check if this can be merged in. Thank you.

@JKorf
Copy link
Copy Markdown
Owner

JKorf commented Feb 6, 2023

Thanks for your work, I had to request a new API key from Zapper, so I'm currently waiting for that before I can test the changes

@nemtajo
Copy link
Copy Markdown
Author

nemtajo commented Mar 21, 2023

Hi Jan. Have you had the time to test the changes?
I've been using GetSupportedAppBalancesAsync, GetAppBalancesAsync and GetTransactionsAsync in production since I made the changes, but the other endpoints I didn't test.

Asking because there will be a major change to balance fetching endpoints, and the old v2/balances will be completely deprecated on 15th of May. It would be good merge this in, and then modify the library for additional changes on balances endpoints (by increasing library version of course).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants