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.

Sign up for GitHub

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

Jump to bottom

feat: add morpho reallocate to #716

Open
wants to merge 21 commits into
base: dev
Choose a base branch
from
Open

feat: add morpho reallocate to #716

wants to merge 21 commits into from

Conversation

halaprix
Copy link
Member

@halaprix halaprix commented Dec 2, 2024 •
edited by zerotucks
Loading

Description of Changes

Please list the changes introduced by this PR:

  • Added MorphoBlueReallocateTo contract for reallocating liquidity to specified markets.
  • Updated configuration files to include Bundler and MorphoBlueReallocateTo entries.
  • Modified operation definitions and strategies to support the new reallocation functionality.
  • Implemented utility functions for fetching and processing market data for reallocation.

How to Test

Please provide instructions on how to test the changes in this PR:

  1. Deploy the updated contracts to a test network.
  2. Configure the environment to use the new MorphoBlueReallocateTo functionality.
  3. Execute a reallocation operation and verify that liquidity is correctly reallocated to the specified market.
  4. Check logs and transaction receipts to ensure no errors occur during the process.

PR Definition of Done

Please ensure the following requirements have been met before marking the PR as ready for review:

  • All checks are passing
  • PR is linked to a corresponding ticket
  • PR title is clear and concise
  • Code has been self-reviewed and any fixes or improvements noted (See Code review standards in Notion)
  • Documentation has been updated if necessary

Feel free to adjust the details as needed to fit your specific changes and testing procedures.

@zerotucks zerotucks marked this pull request as ready for review December 3, 2024 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tomrpl tomrpl tomrpl left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@halaprix @tomrpl @zerotucks