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

📝 Refresh six classic guides #723

Draft
wants to merge 78 commits into
base: main
Choose a base branch
from

Conversation

JFWooten4
Copy link
Contributor

@JFWooten4 JFWooten4 commented Jun 26, 2024

Solving #338 🤝

@JFWooten4 JFWooten4 marked this pull request as draft June 26, 2024 14:12
@JFWooten4
Copy link
Contributor Author

Merge overrode txResponse parse as discussed—thanks for the collaboration! 🫱🏼‍🫲🏻

@JFWooten4
Copy link
Contributor Author

JFWooten4 commented Nov 4, 2024

Re past few weeks, it's been quite a process understanding everyone's perspective to now get back into writing the existing SDEX section's code examples. I'd like to especially thank the above-mentioned community members for both assisting in clarifying the present system and its implications. Upon further outreach I sincerely appreciate @tomerweller, @Row-Bear, @ddombrowsky, Emir, Adam, and Noot whom were instrumental in shaping my current understanding through many generous, collaborative, and thoughtful Discord discussions.

@JFWooten4
Copy link
Contributor Author

JFWooten4 commented Nov 19, 2024

I'm removing the maker/taker diction since it doesn't apply to the DEX. Some of this got added to contextualize Stellar's functions in terms more akin to traditional markets. But these concepts are both outdated and likely unknown for most new readers—and hopefully we can keep it like that. 💡

- By the time this pull merges, the volume/trade approx should hit.
- Fixed missing API doc structure MethodTable explainer.
- Added Lumenauts video to SCP explainer for accessability.
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.

3 participants