Skip to content

docs: fix documentation to reflect Soroban/SEP-41 implementation - #109

Merged
0takuc0mrade merged 1 commit into
Nodus-protocol:mainfrom
barry01-hash:security-md
Jul 21, 2026
Merged

docs: fix documentation to reflect Soroban/SEP-41 implementation#109
0takuc0mrade merged 1 commit into
Nodus-protocol:mainfrom
barry01-hash:security-md

Conversation

@barry01-hash

Copy link
Copy Markdown
Contributor

Closes #106

  • Replace ink!/PSP22/Substrate references with Soroban/SEP-41 equivalents
  • Update threat model to reference Soroban runtime and SEP-41 tokens
  • Update module table to match actual files (liquidity_pool.rs, math.rs, storage.rs, traits.rs, events.rs)
  • Remove references to non-existent lp_token.rs and reentrancy_guard.rs
  • Update contract relationship diagram to use Address instead of AccountId

Summary

Brief description of what this PR does.

Closes #

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Tests
  • Chore / tooling

How Was It Tested?

Describe the tests you ran and how to reproduce them.

Checklist

  • My code follows the style guidelines in CONTRIBUTING.md
  • I have rebased onto the latest main (no merge commits)
  • I have added tests for my changes
  • All existing tests pass locally
  • I have linked the issue above (Closes #N)
  • I have updated documentation if needed

Screenshots (if UI change)

- Replace ink!/PSP22/Substrate references with Soroban/SEP-41 equivalents
- Update threat model to reference Soroban runtime and SEP-41 tokens
- Update module table to match actual files (liquidity_pool.rs, math.rs, storage.rs, traits.rs, events.rs)
- Remove references to non-existent lp_token.rs and reentrancy_guard.rs
- Update contract relationship diagram to use Address instead of AccountId
@barry01-hash

Copy link
Copy Markdown
Contributor Author

Please merge

@0takuc0mrade
0takuc0mrade merged commit 1c3b85f into Nodus-protocol:main Jul 21, 2026
3 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.

docs: ARCHITECTURE.md and SECURITY.md describe ink!/PSP22/Substrate — the contract is Soroban/SEP-41

2 participants