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

PoolTogether connector #73

Merged
merged 24 commits into from
Oct 12, 2021
Merged

Conversation

eccheung4
Copy link
Contributor

Pool Together Connector that will allow DSA to:

  • depositTo : Deposit into a Prize Pool
  • withdrawInstantlyFrom: Withdraw from a Prize Pool
  • claim: Claim from a token faucet
  • claimAll: Claim from multiple token faucet
  • depositToPod: Deposit into a Pod
  • withdrawFromPod: Withdraw shares from Pod

Should also be able to use Uniswap V2 POOL/ETH LP to deposit into PoolTogether Uniswap POOL LP Prize Pool

@thrilok209
Copy link
Contributor

@eccheung4 Can you attach poolTogether docs about the integration?

Also, what does the "claim from a token faucet" function do exactly? Why is it used?

@eccheung4
Copy link
Contributor Author

@thrilok209 Hope these are what you are looking for.

The Token Faucet is Pool Together's mechanism for distributing tokens to user's for joining a Prize Pool. It is an extra incentive for joining a Prize Pool. The Token Faucet drips a certain number of tokens per second. Those tokens are distributed to users based on their balance of measure tokens held. The claim function is used to claim the user's reward from a token faucet.

@eccheung4
Copy link
Contributor Author

@thrilok209 I'm done with edits, let me know if you need anything else when reviewing.

@thrilok209
Copy link
Contributor

@eccheung4 Awesome! We will start reviewing the pr.

@thrilok209
Copy link
Contributor

@eccheung4 Added some suggestions.

@thrilok209
Copy link
Contributor

Hey @eccheung4! We would also need a resolver contract. That is used to fetch protocol and connector functions related data. Resolver helps simplify reading operations on our frontend interfaces.

https://github.com/Instadapp/dsa-resolvers
https://www.notion.so/instadapp/Instadapp-Protocol-Integration-d14f34f1a4aa42138154143948ddb7b4

Eg: Compound resolver- https://github.com/Instadapp/dsa-resolvers/tree/main/contracts/protocols/mainnet/compound

@eccheung4
Copy link
Contributor Author

eccheung4 commented Sep 2, 2021

@thrilok209 wow, thanks for the quick review! Cool, will look into resolver contract. Sorry, didn't know that was needed as well.

@eccheung4
Copy link
Contributor Author

@thrilok209 I believe I've implemented all your suggestions. I'll start on the resolver next.

@eccheung4
Copy link
Contributor Author

@thrilok209 I've submitted PR for resolver and a fix for uniswapStaking. Thanks!

@thrilok209
Copy link
Contributor

@eccheung4 Connector looks good!

Next would be deploying the connector and then adding it to the ConnectorsV2 registry. Could you deploy the connector and comment the deployed connector address in this pr? :)

@eccheung4
Copy link
Contributor Author

eccheung4 commented Sep 24, 2021

@thrilok209 Deployed Connector

ConnectorName: "POOLTOGETHER-A"
ConnectorAddress: 0xbf254fA556703c78d70B4Cec8459d1Ff2b33Cd87

@thrilok209
Copy link
Contributor

@thrilok209 thrilok209 merged commit 3334476 into Instadapp:main Oct 12, 2021
@eccheung4 eccheung4 deleted the pooltogether-connector branch October 26, 2021 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants