Skip to content

feat: Update initialSharePrice initialization logic (Approach 1) #707

Merged
jalextowle merged 88 commits intomainfrom
lm/fix/update-4626-init-logic-3
Dec 19, 2023
Merged

feat: Update initialSharePrice initialization logic (Approach 1) #707
jalextowle merged 88 commits intomainfrom
lm/fix/update-4626-init-logic-3

Conversation

@lucas-manuel
Copy link
Copy Markdown
Contributor

@lucas-manuel lucas-manuel commented Dec 14, 2023

This approach:

  1. Uses a PoolDeployConfig struct in the HyperdriveFactory
  2. Copies all values from that struct into PoolConfig in ERC4626HyperdriveDeployer
  3. Calculates initialSharePrice and adds it to PoolConfig in ERC4626HyperdriveDeployer

Comment thread contracts/src/instances/ERC4626HyperdriveDeployer.sol Outdated
Comment thread contracts/src/instances/ERC4626HyperdriveDeployer.sol
Comment thread contracts/src/instances/ERC4626HyperdriveDeployer.sol Outdated
Copy link
Copy Markdown
Contributor

@jalextowle jalextowle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM once the nits are addressed and the PR is up-to-date.

@lucas-manuel lucas-manuel changed the base branch from main to jalextowle/feature/distribute-excess-idle-overhaul-2 December 18, 2023 22:36
Base automatically changed from jalextowle/feature/distribute-excess-idle-overhaul-2 to main December 19, 2023 00:18
@jalextowle jalextowle enabled auto-merge (squash) December 19, 2023 07:36
@jalextowle jalextowle merged commit 9dba534 into main Dec 19, 2023
@jalextowle jalextowle deleted the lm/fix/update-4626-init-logic-3 branch December 19, 2023 09:09
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.

3 participants