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.

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

Flatten _doJoin #1760

Merged
merged 1 commit into from
Sep 14, 2022
Merged

Flatten _doJoin #1760

merged 1 commit into from
Sep 14, 2022

Conversation

TomAFrench
Copy link
Contributor

builds on #1759

This PR inlines the ManagedPool specific code (LP allowlist and swaps toggle) with the dispatch code.

@TomAFrench TomAFrench added the do not merge Code playground or showcase - not intended to be merged label Sep 14, 2022
Base automatically changed from weighted-pool-joins-lib to master September 14, 2022 15:25
@TomAFrench TomAFrench removed the do not merge Code playground or showcase - not intended to be merged label Sep 14, 2022
@TomAFrench TomAFrench requested a review from nventuro September 14, 2022 15:48
@nventuro nventuro merged commit 8fb97f2 into master Sep 14, 2022
@nventuro nventuro deleted the flatten-do-join branch September 14, 2022 18:42
TomAFrench added a commit that referenced this pull request Sep 14, 2022
* master:
  refactor: flatten _doJoin (#1760)
  Removing return values from VaultActions. (#1765)
TomAFrench added a commit that referenced this pull request Sep 15, 2022
* master:
  Add `ManagedPoolSwapFees` contract (#1755)
  Pull _poolState up into ManagedPool (#1774)
  Adding Foundry support (#1771)
  Cache token weight in _onSwapGivenIn/Out (#1769)
  Fix deployments package (#1773)
  Avoid double zero duration check in `calculateValueChangeProgress` (#1772)
  Extend PoolRegLib (#1770)
  refactor: flatten _doJoin (#1760)
  Removing return values from VaultActions. (#1765)
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.

2 participants