Skip to content

Add UNDERLYING_ADDRESS override for wrapper#17

Open
youngkidwarrior wants to merge 1 commit intochore_wire_anvil_deploy_to_wrapper_create_script_and_set_anvil_signer_via_eoa_deployerfrom
feat_wrapper_add_underlying_address_override_keep_canonical_create_validation_flow
Open

Add UNDERLYING_ADDRESS override for wrapper#17
youngkidwarrior wants to merge 1 commit intochore_wire_anvil_deploy_to_wrapper_create_script_and_set_anvil_signer_via_eoa_deployerfrom
feat_wrapper_add_underlying_address_override_keep_canonical_create_validation_flow

Conversation

@youngkidwarrior
Copy link
Copy Markdown
Collaborator

Why:
Allow overriding the underlying token address while keeping
canonical/create validation flow.

Test plan:

  • Run scripts/wrapper/create.ts with override.
  • Verify underlying matches configured address.

Copy link
Copy Markdown
Collaborator Author

youngkidwarrior commented Sep 10, 2025

@youngkidwarrior youngkidwarrior marked this pull request as ready for review September 10, 2025 00:55
@youngkidwarrior youngkidwarrior changed the base branch from feat_deploy_require_existing_sendx_pass_to_rewardsmanager_persist_sendx_in_deployments to graphite-base/17 September 12, 2025 11:07
@youngkidwarrior youngkidwarrior force-pushed the feat_wrapper_add_underlying_address_override_keep_canonical_create_validation_flow branch from f44d3af to d7ea47b Compare September 12, 2025 11:08
@youngkidwarrior youngkidwarrior changed the base branch from graphite-base/17 to chore_wire_anvil_deploy_to_wrapper_create_script_and_set_anvil_signer_via_eoa_deployer September 12, 2025 11:08
…ndearnrewards_bubble_erc4626_hooks_factory-gated_join_flow (886f744)

Why:
Apply the same non-artifacts changes from top-of-stack to this branch:
- Rename RewardsAggregator -> SendEarnRewards
- Bubble ERC4626 hooks; factory-gated vault routing
- Ignition module rename; minimal tests/mocks; docs labels

Test plan:
- npx hardhat compile
- npx hardhat test --grep "SendEarnRewards"
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