-
Notifications
You must be signed in to change notification settings - Fork 5
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
Feat/uniswapv3 #39
Feat/uniswapv3 #39
Conversation
Will still need to add a link to the UniV3 dashboard url once it is up and running. |
V3 dashboard links have been added with 835e1fc |
grammar
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.
Checked for grammar and spelling errors.
docs/build-with-powerloom/use-cases/existing-implementations/uniswap-dashboard/index.md
Outdated
Show resolved
Hide resolved
docs/build-with-powerloom/use-cases/existing-implementations/uniswap-dashboard/data-points.md
Outdated
Show resolved
Hide resolved
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.
LGTM after resolving open conversations.
Fixes #
Checklist
Current behaviour
There are no references to the UniswapV3 use case in the docs.
New expected behaviour
The UniswapV2 existing use case section has been changed to cover all Uniswap use cases including V3.
Change logs
Added
Changed
Deployment Instructions
No changes