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

CoW AMM: use useManagedTransaction hook to prepare for EOA transactions in AMM app #701

Conversation

ribeirojose
Copy link
Contributor

Copy link

linear bot commented Jun 10, 2024

Copy link

vercel bot commented Jun 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
balancer-tools ❌ Failed (Inspect) Jun 11, 2024 9:13pm
chainlink-data-feeds-api ❌ Failed (Inspect) Jun 11, 2024 9:13pm
cow-amm-deployer ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 11, 2024 9:13pm
cow-tools ❌ Failed (Inspect) Jun 11, 2024 9:13pm

yvesfracari and others added 3 commits June 10, 2024 13:28
* wip: refactor page structure and add amms page

* change page to async without reload button

* merge into base branch

* refactor create amm form file structure and use useWatch hook

* Create form page wrapper and create edit form

* remove unecessary div on form wrapper

* refactor fetch data to use same interface on all requests part2

* refactor alercard layout

* refactor: link component look for disable proporty of children

* refactor disable transaction for old version and add migrate args builder

* add old version alert and trigger tx

* Add tx pending alert when tx interacting with the cowamm (#698)

* chore: add fetch has amm tx pending

* chore: add pendig amm tx card

* chore: add tx pending alert card on my amms page

* Add error on creating amm that already exists check token and refactor form schemas (#699)

* add validation to check if amm exists

* refactor: connect form types with ICoWAMM

* Create add liquidity flow (#700)

* add deposit page UI with schema validations

* add deposit transaction
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