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

feat(inverse-firm): Add cvxFxs rewards #2861

Merged
merged 3 commits into from Aug 1, 2023

Conversation

webmass
Copy link
Contributor

@webmass webmass commented Aug 1, 2023

Description

Add the CVX+FXS rewards for the cvxFXS FiRM market

Checklist

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

How to test?

Via the inverse website:
For the user 0xAAc0aa431c237C2C0B5f041c8e59B3f1a43aC78F rewards should match with what's displayed here (need wallet connected): https://www.inverse.finance/firm/cvxFXS?viewAddress=0xAAc0aa431c237C2C0B5f041c8e59B3f1a43aC78F

Via etherscan:
For the user 0xAAc0aa431c237C2C0B5f041c8e59B3f1a43aC78F rewards should match with
the claimableRewards read function (the escrow address to use for the call for that user: 0xE873174F2988c4C1eEE3D820b30F1f8cD24cA220) so claimableRewards(0xE873174F2988c4C1eEE3D820b30F1f8cD24cA220)
https://etherscan.io/address/0x49b4d1dF40442f0C31b1BbAEA3EDE7c38e37E31a#readContract

@wpoulin wpoulin changed the title Firm: add cvxFxs rewards feat(inverse-firm): Add cvxFxs rewards Aug 1, 2023
@wpoulin
Copy link
Contributor

wpoulin commented Aug 1, 2023

Looks good ! Rename the last commit message to feat(inverse-firm): Add cvxFxs rewards and CI will be 🟢 @webmass

@wpoulin wpoulin merged commit 31c116d into Zapper-fi:main Aug 1, 2023
6 checks passed
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

2 participants