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

Update DEX.md #97

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Update DEX.md #97

wants to merge 1 commit into from

Conversation

Honzan9
Copy link
Collaborator

@Honzan9 Honzan9 commented Mar 14, 2023

Hi @0ptim, This is the first version of the DEX page which was composed from the previous content. It is not finished yet, cause I wanted to add a chapter about DEX mechanics (formula of pool ratio, slipage, etc), but now I'm not sure, maybe would be better to make separate page for this? What do you mean?

The first version of the DEX page which was composed from the previous content.
@Honzan9
Copy link
Collaborator Author

Honzan9 commented Mar 14, 2023

The build failed because the links are pointing to the version of sidebars.js in Chapter reorganization PR, after merge should be solved.

@0ptim
Copy link
Owner

0ptim commented Mar 15, 2023

Hey @Honzan9
Difficult for me to know if it's better to split or not. Just do what you think is best. The content itself is most important and, if necessary, we can always split at a later stage.

@0ptim
Copy link
Owner

0ptim commented Mar 15, 2023

It's okay if the build fails, as long as you are in draft mode. This way you have instant feedback, and it's just for you.

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