Skip to content

Conversation

@SynnekOG
Copy link

PR Description

Summary

This PR extends the LiquidityManagerV2 contract by introducing internal helper functions to manage liquidity provider state and vesting schedules.

Changes

  • Added _updateLiquidityProvider internal function:
    • Updates provider data (amount, timestamp, liquidity).
    • Adds new providers to the pool.
    • Triggers vesting setup once liquidity thresholds are met.
    • Emits LiquidityThresholdReached event.
  • Added _setupVesting internal function:
    • Configures vesting schedules via vestingContract.
    • Emits VestingScheduleCreated event.
  • Applied forge fmt for consistent code formatting.

Impact

  • Improves modularity and maintainability by separating provider updates and vesting logic.
  • Enables automated vesting schedule setup once thresholds are reached.
  • Provides richer event logs for external monitoring.

Next Steps

  • Write unit tests for _updateLiquidityProvider and _setupVesting.
  • Validate event emissions in integration tests.
  • Extend vesting logic to support multiple token types if required.

@2PykeETH 2PykeETH self-requested a review September 22, 2025 15:55
@2PykeETH 2PykeETH added documentation Improvements or additions to documentation enhancement New feature or request labels Sep 22, 2025
@2PykeETH 2PykeETH merged commit 6ec371a into CraftMeme:main Sep 22, 2025
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants