Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

feat: add olympus V2 bond integration #203

Merged
merged 11 commits into from Apr 23, 2022
Merged

Conversation

brightiron
Copy link
Contributor

Description

For review/feedback by Zapper team. Setting as WIP for now.

Includes returning/displaying wallet olympus bond contract balances split by currently vesting and currently claimable.

@brightiron brightiron changed the title add olympus bond integration feat: add olympus bond integration Apr 22, 2022
@JForsaken
Copy link
Contributor

Hey @brightiron thanks for contributing! We already have Olympus as an app integration, but it's currently in our internal integrations -- let me move it to the Studio repo so we can check together what are the missing parts!

@brightiron brightiron marked this pull request as ready for review April 23, 2022 07:19
@brightiron
Copy link
Contributor Author

@immasandwich @JForsaken - I merged main back into this branch and reimplemented based on the existing patterns.

This PR adds support for Olympus V2 Bonds and will display both vesting and claimable gOHM in the bond depository contract for a specific wallet address.

Please by all means feel free to improve on this PR. I had limited time and wasn't able to really dig deep into this codebase.

Thx!

@brightiron brightiron changed the title feat: add olympus bond integration feat: add olympus V2 bond integration Apr 23, 2022
@immasandwich immasandwich merged commit cbde9d4 into Zapper-fi:main Apr 23, 2022
@SamIam-0x
Copy link
Contributor

hello @appleseed-iii and @brightiron !!

Looks like you did not provide an address when submitting your PR. I recognize it may have seemed kind of random that we were asking for a wallet address with a PR submission, but we are actually looking for ways to reward Studio contributors in the future.

Is there an address you’d want to provide? Could even be an empty address, if you do not have an doxxed address

Let me know! Also, if you’d like to privately provide it, feel free to shoot me a TG https://t.me/SamIAm_0x or DM on the Zapper Discord.

Thanks!
Sam

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.

None yet

5 participants