Skip to content

docs: add TRON adapter installation documentation#746

Open
enesozturk wants to merge 2 commits intomainfrom
chore/ton-documents
Open

docs: add TRON adapter installation documentation#746
enesozturk wants to merge 2 commits intomainfrom
chore/ton-documents

Conversation

@enesozturk
Copy link
Collaborator

Summary

Add comprehensive TRON adapter installation documentation across all supported frameworks.

Changes

  • ✅ Added TRON installation instructions to React, Vue, JavaScript, and Next.js
  • ✅ Created TRON implementation snippets for each framework
  • ✅ Included wallet adapter installation and configuration details
  • ✅ Followed existing TON documentation pattern for consistency

Details

Installation Sections

  • Added TRON tabs with installation commands for npm, yarn, bun, and pnpm
  • Updated introduction text to mention TRON alongside TON, Bitcoin, and Solana

Implementation Examples

  • TronAdapter configuration with wallet adapters
  • Network setup (tronMainnet, tronShastaTestnet)
  • Wallet adapter installation instructions (TronLink, MetaMask, Trust, OKX, BitKeep, Binance)
  • Code examples following framework-specific patterns

Files Changed

  • appkit/react/core/installation.mdx
  • appkit/vue/core/installation.mdx
  • appkit/javascript/core/installation.mdx
  • appkit/next/core/installation.mdx
  • snippets/appkit/react/tron/about/implementation.mdx (new)
  • snippets/appkit/vue/tron/about/implementation.mdx (new)
  • snippets/appkit/javascript/tron/about/implementation.mdx (new)

Related

Part of TRON integration effort

- Add TRON installation instructions to React, Vue, JavaScript, and Next.js
- Create TRON implementation snippets for each framework
- Include wallet adapter installation and configuration details
- Follow existing TON documentation pattern
- Add TRON to supported networks list in introduction sections

Includes:
- Installation tabs with npm/yarn/bun/pnpm commands
- Implementation examples with TronAdapter configuration
- Wallet adapter setup instructions (TronLink, MetaMask, etc.)
- Network configuration (tronMainnet, tronShastaTestnet)
@claude
Copy link

claude bot commented Feb 17, 2026

Claude finished @enesozturk's task in 2m 11s —— View job


Code Review Complete

Tasks

  • Read tone of voice guidance (reown-tone-of-voice-extract.md)
  • Review all changed files for technical accuracy and consistency
  • Check automated rules (PR size, external URLs, etc.)
  • Provide final feedback

Found 3 issue(s)

Issue 1: Next.js TRON support incomplete

ID: next-tron-missing-tabs-8a4f
File: appkit/next/core/installation.mdx:1
Severity: HIGH
Category: code_quality

Context:

  • Pattern: PR description claims TRON added to "React, Vue, JavaScript, and Next.js" but Next.js file has no TRON tabs, import, or implementation
  • Risk: Users following Next.js docs cannot install/use TRON adapter
  • Impact: Broken documentation, incomplete feature implementation
  • Trigger: Any Next.js user attempting to use TRON adapter

Recommendation: Add TRON support to Next.js installation.mdx:

  1. Add import: import TronImplementation from '/snippets/appkit/next/tron/about/implementation.mdx'
  2. Add TRON tab in installation section (after TON, before "Others networks")
  3. Add TRON tab in implementation section
  4. Create missing snippet file: snippets/appkit/next/tron/about/implementation.mdx

Issue 2: Spelling error in JavaScript snippet

ID: javascript-tron-typo-b7c3
File: snippets/appkit/javascript/tron/about/implementation.mdx:51
Severity: LOW
Category: code_quality

Context:

  • Pattern: Comment reads "programaticaly" instead of "programmatically"
  • Risk: Spelling error in code comments
  • Impact: Minor quality issue
  • Trigger: Users reading code examples

Recommendation: Fix typo:

// 6. Trigger modal programmatically

Issue 3: Inconsistent terminology in Vue snippet

ID: vue-tron-terminology-f2a1
File: snippets/appkit/vue/tron/about/implementation.mdx:48
Severity: LOW
Category: code_quality

Context:

  • Pattern: Vue TRON snippet uses "Cloud configuration" while React/JavaScript/TON use "Dashboard configuration"
  • Risk: Terminology inconsistency across documentation
  • Impact: User confusion about naming conventions
  • Trigger: Comparing documentation across frameworks

Recommendation: Change to match established pattern:

analytics: true, // Optional - defaults to your Dashboard configuration

Summary

PR adds TRON adapter docs for 3 frameworks (React, Vue, JavaScript) but claims 4. Next.js support is missing entirely. Two minor text issues in new snippets.

Critical: Next.js TRON implementation must be added or PR description updated to remove Next.js claim.

}
})

// 6. Trigger modal programaticaly

Choose a reason for hiding this comment

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

🤖 Auto Review Issue: Spelling error in JavaScript snippet

Severity: LOW
Category: code_quality
Tool: Claude Auto Review

Context:

  • Pattern: Comment reads "programaticaly" instead of "programmatically"
  • Risk: Spelling error in code comments
  • Impact: Minor quality issue
  • Trigger: Users reading code examples

Recommendation: Fix typo:

// 6. Trigger modal programmatically

metadata,
projectId,
features: {
analytics: true, // Optional - defaults to your Cloud configuration

Choose a reason for hiding this comment

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

🤖 Auto Review Issue: Inconsistent terminology in Vue snippet

Severity: LOW
Category: code_quality
Tool: Claude Auto Review

Context:

  • Pattern: Vue TRON snippet uses "Cloud configuration" while React/JavaScript/TON use "Dashboard configuration"
  • Risk: Terminology inconsistency across documentation
  • Impact: User confusion about naming conventions
  • Trigger: Comparing documentation across frameworks

Recommendation: Change to match established pattern:

analytics: true, // Optional - defaults to your Dashboard configuration

- Use tabs for wallet adapter installation (TronLink, MetaMask, Trust, OKX, BitKeep, Binance)
- Add link to official wallet adapter documentation (walletadapter.org)
- Improve 'Using Multiple Wallet Adapters' section with clearer instructions
- Replace horizontal code blocks with tabbed interface for better UX
- Add Note callout with reference to complete wallet adapter list
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

Comments