Skip to content

Refactor Farming and DEX contracts for improved functionality and cla…#8

Merged
jamesatomc merged 1 commit intomasterfrom
dex2
Sep 13, 2025
Merged

Refactor Farming and DEX contracts for improved functionality and cla…#8
jamesatomc merged 1 commit intomasterfrom
dex2

Conversation

@jamesatomc
Copy link
Owner

…rity

  • Updated FarmingPage component to use new contract functions: totalSupply, getRewardRate, and isPaused.
  • Modified contract addresses in contracts.ts for KANARI, DEX_FACTORY, KANARI_NATIVE_POOL, and FARMING.
  • Enhanced Farming contract ABI with new functions: getPeriodFinish, getRewardRate, getRewardForDuration, and isPaused.
  • Adjusted DeployDEX script to mint rewards directly to the fee recipient and ensure proper funding of the farming contract.
  • Improved internal functions in ConstantProductAMM and Farming contracts for better readability and precision handling.
  • Updated tests in Farming.t.sol to reflect changes in reward rate calculations and getter functions.

…rity

- Updated FarmingPage component to use new contract functions: totalSupply, getRewardRate, and isPaused.
- Modified contract addresses in contracts.ts for KANARI, DEX_FACTORY, KANARI_NATIVE_POOL, and FARMING.
- Enhanced Farming contract ABI with new functions: getPeriodFinish, getRewardRate, getRewardForDuration, and isPaused.
- Adjusted DeployDEX script to mint rewards directly to the fee recipient and ensure proper funding of the farming contract.
- Improved internal functions in ConstantProductAMM and Farming contracts for better readability and precision handling.
- Updated tests in Farming.t.sol to reflect changes in reward rate calculations and getter functions.
@vercel
Copy link

vercel bot commented Sep 13, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
sm Ready Ready Preview Comment Sep 13, 2025 10:53am

@jamesatomc jamesatomc merged commit b55cc00 into master Sep 13, 2025
2 of 4 checks passed
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