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

V4 create pool #788

Merged
merged 3 commits into from
Oct 11, 2024
Merged

V4 create pool #788

merged 3 commits into from
Oct 11, 2024

Conversation

saucepoint
Copy link
Collaborator

No description provided.

Copy link

vercel bot commented Oct 3, 2024

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

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 8, 2024 3:01pm

@saucepoint saucepoint requested a review from akarys92 October 3, 2024 20:32
Copy link
Collaborator

@akarys92 akarys92 left a comment

Choose a reason for hiding this comment

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

Comments below!

@@ -2,6 +2,165 @@
title: Create Pool
---

## 🚧 Under construction 🚧
### Context
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why start with an H3? This renders weird to my eye on the actual page. Maybe should be an H2 since it's the first header?

Please see https://www.v4-by-example.org/initialize
Creating a pool on Uniswap v4 is permissionless and enables the trading of an asset.

Below, there are two approaches to creating a pool:
Copy link
Collaborator

Choose a reason for hiding this comment

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

We should flesh this intro out a bit more. Why would a developer want to create a pool? At the end of this guide what will be true that isn't when they started? Any pre-reqs they should be aware of? Feels abrupt to just jump in like this.

forge install uniswap/v4-periphery
```

## Guide: Create a Pool Only
Copy link
Collaborator

Choose a reason for hiding this comment

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

Would be good to clarify in what case would someone create a pool with no liquidity?

```
> Excess Ether is **NOT** refunded unless developers encoded `SWEEP` in the `actions` parameter

For a full end-to-end script, developers should see [v4-template#script](https://github.com/uniswapfoundation/v4-template/tree/main/script)
Copy link
Collaborator

Choose a reason for hiding this comment

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

👀 that format again

@akarys92 akarys92 merged commit fad2630 into main Oct 11, 2024
5 checks passed
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