Skip to content

chore: update contracts#63

Open
miguel-nascimento wants to merge 2 commits into
mainfrom
miguel/update-contracts
Open

chore: update contracts#63
miguel-nascimento wants to merge 2 commits into
mainfrom
miguel/update-contracts

Conversation

@miguel-nascimento

@miguel-nascimento miguel-nascimento commented Dec 11, 2024

Copy link
Copy Markdown
Contributor

we probably want to wait the new flow to be ready to merge all together

  • but I decided to open this PR so theo can have a reference about how to set up the rewards distribution contract

I'm unsure about the claim flow, since I couldn't test it (I dont have river balance to claim)

  • anyway, we will polish it in the next days

Tested delegate & authorize claimer with address zero.

@vercel

vercel Bot commented Dec 11, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
river-build-www ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 11, 2024 2:26am
www ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 11, 2024 2:26am

Comment thread src/contracts.ts

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ideally, this file should be git ignored -- I can't remember if we had an special reason for not ignoring it

Comment thread wagmi.config.ts
Comment on lines +1347 to +1349
const mainnetRiverTokenAddress = '0x53319181e003E7f86fB79f794649a2aB680Db244' satisfies Address
const baseRiverTokenAddress = '0x91930fd11ABAa5241241d3B07c02A8d0B5ac1920' satisfies Address
const baseGammaSepoliaTokenAddress = '0x24e3123E1b30E041E2df26Da9d6140c5B07Fe4F0' satisfies Address

@miguel-nascimento miguel-nascimento Dec 11, 2024

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

in the next @river-build/generated version, we can replace this with values from npm

Comment thread wagmi.config.ts
Comment on lines -2013 to 1381
address: {
[mainnet.id]: '0x53319181e003e7f86fb79f794649a2ab680db244',
[sepolia.id]: '0x40ef1bb984503bb5adef041a88a4f9180e8586f9',
},
},
{
name: 'RiverAuthorizer',
name: 'Authorizer',
address: {
[mainnet.id]: '0x0bEe55b52d01C4D5d4D0cfcE1d6e0baE6722db05',
[sepolia.id]: '0x2f5E8F6Fb7EcF63d13C13B698d1e0B3EA4Ef604B',

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

unsure if that's the most up-to-date address of the authorizer contract - if it is, we should document it in the river docs - only found one reference matching this mainnet address in the river repo (hardcoded in a solidity file - not even inside /contracts or /packages/generated)

Comment thread wagmi.config.ts
},
}
abi: rewardDistributionAbi,
},
],
plugins: [
etherscan({

@miguel-nascimento miguel-nascimento Dec 11, 2024

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

would love to import abi from npm instead of using etherscan plugin or manually hardcoding abi in this app

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant