Skip to content

refactor: custom decimals - solady - factory#28

Open
hexshire wants to merge 4 commits intomainfrom
dev
Open

refactor: custom decimals - solady - factory#28
hexshire wants to merge 4 commits intomainfrom
dev

Conversation

@hexshire
Copy link
Member

@hexshire hexshire commented Mar 31, 2025

  1. Migration to Solady:

    • Replaced OpenZeppelin dependencies with their Solady equivalents
    • Changed SafeERC20 and SafeCast from OpenZeppelin to Solady versions
    • Updated interfaces and types to use Solady versions
  2. Factory Changes:

    • Modified deployXERC20 function to include _decimals parameter
    • New deployXERC20WithLockbox function that combines XERC20 and Lockbox deployment in a single transaction
    • Removed separate deployLockbox function
    • Updated deployment logic to use CREATE3.deployDeterministic
  3. XERC20 Changes:

    • Added decimals parameter in constructor
    • Updated ownership logic to use Solady version
    • Improved error handling and events
  4. Lockbox Changes:

    • Simplified ERC20 interface using address instead of IERC20
    • Updated transfer functions to use Solady versions
    • Improved native token handling
  5. Test Updates:

    • Adapted tests to use new Solady interfaces
    • Updated mocks and expectations
    • Improved test coverage
  6. Configuration Improvements:

    • Updated deployment configuration file to include decimals
    • Improved documentation and comments
  7. Internal Review Fixes

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