refactor: Update zapper's endpoints - Githubissues

yearn / yearn-sdk

🦧 SDK for the yearn.finance platform. WIP

https://npm.im/@yfi/sdk

MIT License

53 stars 56 forks source link

refactor: Update zapper's endpoints #284

Closed karelianpie closed 2 years ago

karelianpie commented 2 years ago

Description

Use the new zapper endpoints.

v2/apps/tokens/balances now has the addresses inside "balances" (https://github.com/yearn/yearn-sdk/pull/284/commits/4e3371078c6681cd67390ccc9e34eb66ec12832c addresses that)

Related Issue

https://medium.com/zapper-protocol/updates-to-zappers-public-apis-4933a2967f57

Motivation and Context

Old endpoints have been deprecated

How Has This Been Tested?

Hit the v2 endpoints to make sure the results were the same as the v1;
Yalc'ed the SDK into the FE;
Added src/utils/encode.spec.ts to test the base64 encoding.

Screenshots (if appropriate):

Screen Shot 2022-04-26 at 8 28 16 pm

... more in the files

github-actions[bot] commented 2 years ago

size-limit report 📦

Path	Size
dist/sdk.cjs.production.min.js	46.95 KB (+0.13% 🔺)
dist/sdk.esm.js	47.32 KB (+0.12% 🔺)

karelianpie commented 2 years ago

What do we think about using https://github.com/colinhacks/zod here?