-
Notifications
You must be signed in to change notification settings - Fork 199
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.
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
Feature/liquidity provider sandbox #16
Conversation
packages/asset-swapper/src/utils/market_operation_utils/sampler_operations.ts
Show resolved
Hide resolved
packages/asset-swapper/src/utils/market_operation_utils/sampler_operations.ts
Show resolved
Hide resolved
packages/asset-swapper/src/quote_consumers/exchange_proxy_swap_quote_consumer.ts
Outdated
Show resolved
Hide resolved
packages/asset-swapper/src/quote_consumers/exchange_proxy_swap_quote_consumer.ts
Outdated
Show resolved
Hide resolved
contracts/zero-ex/contracts/src/features/ILiquidityProviderFeature.sol
Outdated
Show resolved
Hide resolved
contracts/zero-ex/contracts/src/features/ILiquidityProviderFeature.sol
Outdated
Show resolved
Hide resolved
public | ||
FixinCommon() | ||
{ | ||
weth = weth_; | ||
sandbox = new LiquidityProviderSandbox(zeroEx); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't we rather pass in the sandbox so we can reuse an older one and avoid having to whitelist a new one every time we make a change to this feature?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah I was thinking in the next iteration of this feature, sandbox
would be passed in as a constructor arg. no need to pass it in for this first version though, might as well deploy it in the same txn
think a rebase will fix the |
8849fc3
to
ca8e066
Compare
b93330e
to
c4458bb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👏 👏 👏
Has this been simbotted btw? |
ace05ce
to
32e1bd0
Compare
d09a6ab
to
b40eee5
Compare
b40eee5
to
06295c4
Compare
This reverts commit 7403c02.
This reverts commit 7403c02.
Description
ILiquidityProvider
interfaceProtocolFeeUnfunded
event in FQTTesting instructions
Types of changes
Checklist:
[WIP]
if necessary.