Skip to content

Amm approve erc20 on setup #1762

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

Merged
merged 2 commits into from
Sep 18, 2020
Merged

Amm approve erc20 on setup #1762

merged 2 commits into from
Sep 18, 2020

Conversation

dong77
Copy link
Contributor

@dong77 dong77 commented Sep 18, 2020

No description provided.

@dong77 dong77 requested a review from Brechtpd September 18, 2020 04:08
@@ -65,9 +65,7 @@ contract AmmPool is IBlockReceiver {
uint pos
);

uint public constant MAX_UINT = ~uint(0);
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not use a constant?

@Brechtpd Brechtpd merged commit a289c3c into protocol36-ammpool Sep 18, 2020
@Brechtpd Brechtpd deleted the amm-approve-erc20 branch September 18, 2020 14:44
Brechtpd added a commit that referenced this pull request Sep 22, 2020
* [protocol 3.6] AMM pool testing

* [protocol 3.6] More AMM pool testing

* [protocol 3.6] Improved AMM pool + testing

* [protocol 3.6] Small optimizations

* [protocol 3.6] Misc improvements/fixes AMM pool

* unify submitBlocksCompressedWithCallbacks (#1761)

* unify submitBlocksCompressedWithCallbacks

* unify submitBlocksCompressedWithCallbacks

* Amm approve erc20 on setup (#1762)

* approve upon setup

* approve upon setup

* allow withdraw all tokens (#1763)

* make agent explicitly inherit IAgent (#1765)

* [protocol 3.6] Small fixes

* [protocol 3.6] uint96 safe math (#1766)

* [AMM Pool] Rename variables and extract methods (#1772)

* rename variables

* extract methods

* extract methods

* [AMM Pool] Some small improvements (#1773)

* add changes based on my questions

* more

Co-authored-by: wangdong <wangdong@zhongan.io>
Co-authored-by: Brechtpd <Brechtp.Devos@gmail.com>

* [AMM pool] Changed approvals + locking logic (#1771)

* [protocol 3.6] Changed approvals + locking logic

* [protocol 3.6] LP tokens

* [protocol 3.6] Use minPoolAmountOut for joins

* [protocol 3.6] AMM pool signature assisted L1 withdrawals + fixes

* [protocol 3.6] Small improvements/fixes

Co-authored-by: Daniel Wang <daniel@loopring.org>
Co-authored-by: wangdong <wangdong@zhongan.io>
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