Skip to content

Remove DEX page from nav and fix multi-sig cross-reference#622

Open
abn2357 wants to merge 1 commit into
tronprotocol:masterfrom
abn2357:dex_multi-signatures
Open

Remove DEX page from nav and fix multi-sig cross-reference#622
abn2357 wants to merge 1 commit into
tronprotocol:masterfrom
abn2357:dex_multi-signatures

Conversation

@abn2357

@abn2357 abn2357 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove the outdated DEX page from site navigation. The native Bancor exchange is outdated compared to modern DEXes and is effectively non-functional since ExchangeTransactionContract is disabled; the source file is kept in the repo for reference.
  • Fix a stale cross-reference in multi-signatures.md: the ContractType overview table moved to system-contracts.md, so the AVAILABLE_CONTRACT_TYPE note now links to the relocated table instead of referencing a non-existent local table.

Test plan

  • mkdocs.yml nav no longer lists the DEX page; the entry is removed cleanly with valid YAML
  • multi-signatures.md link resolves to the #contracttype-overview anchor in system-contracts.md

The native Bancor exchange is outdated compared to modern DEXes and is
effectively non-functional since ExchangeTransactionContract is disabled,
so drop it from the site navigation; the source file is kept in the repo
for reference. Also fix a stale cross-reference in multi-signatures.md:
the ContractType overview table moved to system-contracts.md, so the
AVAILABLE_CONTRACT_TYPE note now links to the relocated table instead of
referencing a non-existent local table.

@vivian1912 vivian1912 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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