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

feat(concave): Add Concave LSDCNV Integration #1424

Merged
merged 5 commits into from
Sep 24, 2022
Merged

feat(concave): Add Concave LSDCNV Integration #1424

merged 5 commits into from
Sep 24, 2022

Conversation

ibeandyy
Copy link
Contributor

@ibeandyy ibeandyy commented Sep 15, 2022

Description

Added support for Concave staking contract

Checklist

  • I have followed the Contributing Guidelines
  • (optional) As a contributor, my Ethereum address/ENS is: ibeandyy.eth
  • (optional) As a contributor, my Twitter handle is: @ibeandyy

How to test?

/apps/concave/balances?addresses[]=0x3c1c5DD60D1C28Ca1adf907b08737EDedF6D29fB&network=ethereum

/apps/concave/positions?groupIds[]=lsdcnv&network=ethereum

@@ -0,0 +1,28 @@
import axios from 'axios';
export const getuserPositions = async () => {
const gqlUrl = 'https://concave.hasura.app/v1/graphql';
Copy link
Contributor

Choose a reason for hiding this comment

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

Hey @ibeandyy, most of your PR looks good but I'm a little nervous about this part here since I don't think the GQL server will stand up to our traffic. I know what you're trying to do though, and I think we can get the Lock events directly from our Ethers instance. I'll merge your PR and try to make that change (its not documented, so I don't want you to have to struggle with figuring out on your own 🙏 )

@immasandwich immasandwich merged commit c14d1d3 into Zapper-fi:main Sep 24, 2022
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