-
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
add description of DEX #53
add description of DEX #53
Conversation
happy to review this and #55, just ping me for review! |
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.
content looks good to me, left some feedback 👍
I think at some point @smmhrdmn may want to redesign some of the graphics across the guide (?), and these serve as good references for shaping the information that will be embedded in those designs.
totally! this would have ideally been a part of the guide redesign but we never got there. these are great additions, and as you say we'll now have a better reference to go off of |
thanks for review @TalDerei! made some changes |
LGTM! |
ac9f6a7
to
0ace71a
Compare
heads up I don't have merge access in this repo |
cc @conorsch |
You do now! Have at it! |
Adds a description of the DEX
Also moves up the liquidity provisioning section in the sidebar
Comments welcome!