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

Feat/v2 xerc20 #44

Open
wants to merge 12 commits into
base: feat/v2
Choose a base branch
from
Open

Feat/v2 xerc20 #44

wants to merge 12 commits into from

Conversation

KristenPire
Copy link
Contributor

@KristenPire KristenPire commented Jul 2, 2024

This pull request introduces significant changes to the token minting and allowance system by replacing the MintAllowanceUpgradeable contract with a new RateLimitsUpgradeable contract. Additionally, it includes various updates to deployment scripts and contract configurations to align with the new rate limits approach.

Key Changes:

Major Contract Updates:

  • Rate Limits Implementation:

    • Replaced MintAllowanceUpgradeable with RateLimitsUpgradeable to manage minting and burning limits more dynamically (src/RateLimitsUpgradeable.sol).
    • Updated Token contract to use RateLimitsUpgradeable instead of MintAllowanceUpgradeable (src/Token.sol). [1] [2] [3]
  • Contract Adjustments:

    • Modified various controller contracts (ControllerToken, EthereumControllerToken, GnosisControllerToken, PolygonControllerToken) to use _useMinterLimits instead of _useMintAllowance for checking minting permissions. [1] [2] [3] [4]

Deployment and Configuration Scripts:

  • Script Updates:

    • Updated configureToken.s.sol to use LIMIT_CAP instead of MAX_MINT_ALLOWANCE and added new logic for setting rate limits (script/configureToken.s.sol). [1] [2]
    • Added support for Gnosis Mainnet deployment in deployAll.sh (script/deployAll.sh). [1] [2]
    • Updated shell script to use LIMIT_CAP environment variable (script/configureToken.sh).
  • Batch Mint Script:

    • Commented out the setMaxMintAllowance and setMintAllowance calls in the batchMint.s.sol script, indicating a shift away from the old allowance system (script/batchMint.s.sol).

These changes collectively enhance the flexibility and security of the token minting process by introducing rate limits and updating the deployment scripts to support the new system.

@KristenPire KristenPire self-assigned this Jul 2, 2024
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