Skip to content

feat: initialize Next.js Tron application with essential configuratio…#206

Merged
rtomas merged 1 commit into
mainfrom
rtomas/next-tron-example
May 24, 2026
Merged

feat: initialize Next.js Tron application with essential configuratio…#206
rtomas merged 1 commit into
mainfrom
rtomas/next-tron-example

Conversation

@rtomas

@rtomas rtomas commented May 24, 2026

Copy link
Copy Markdown
Collaborator

…ns and components

@vercel

vercel Bot commented May 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
appkit-web-examples-javascript-bitcoin Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-javascript-ethers Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-javascript-solana Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-javascript-wagmi Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-next-bitcoin Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-next-ethers Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-next-multichain Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-next-solana Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-next-wagmi Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-react-bitcoin Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-react-core-up Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-react-ethers Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-react-ethers5 Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-react-multichain Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-react-solana Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-react-wagmi Building Building Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-vue-bitcoin Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-vue-ethers Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-vue-multichain Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-vue-solana Ready Ready Preview, Comment May 24, 2026 7:42pm
appkit-web-examples-vue-wagmi Ready Ready Preview, Comment May 24, 2026 7:42pm
react-core-universal-connector Ready Ready Preview, Comment May 24, 2026 7:42pm

Request Review

@rtomas rtomas merged commit d858acf into main May 24, 2026
4 of 26 checks passed
@claude

claude Bot commented May 24, 2026

Copy link
Copy Markdown

Claude finished @rtomas's task in 2m 14s —— View job


Code Review

  • Gather context and read changed files
  • Run license compliance check
  • Run breaking changes check
  • Run data classification check
  • Compile and post findings

🔄 Full Review — new example project nextjs/next-tron-app-router (20 files, 511 lines added)

Subagent results:

  • License compliance: ✅ No issues — only new dep is @tronweb3/tronwallet-adapter-tronlink (MIT)
  • Breaking changes: ✅ No issues — purely additive, no existing files modified
  • Data classification: ✅ No issues — hardcoded projectId is a public demo key, established repo pattern
Found 1 issue(s)

Issue 1: External domain URLs in metadata

ID: context-external-domain-urls-a3f1
File: nextjs/next-tron-app-router/src/context/index.tsx:12-13
Severity: LOW
Category: code_quality

🔒 External Domain URL (Non-blocking)

Context:

  • Pattern: Metadata url points to https://github.com/0xonerb/next-reown-appkit-ssr — a third-party GitHub repo, not a Reown domain. The icons field also uses https://avatars.githubusercontent.com/u/179229932.
  • Risk: The url field in AppKit metadata is used for domain verification (origin matching). Pointing to a third-party repo is incorrect for a Reown official example — it should reflect the example's actual deployed URL or a placeholder using a Reown domain.
  • Impact: Domain verification mismatch; copied code could silently use wrong origin.
  • Trigger: Present at runtime for all users connecting wallets.

Recommendation:

const metadata = {
  name: 'next-reown-appkit',
  description: 'next-reown-appkit',
  url: 'https://reown.com', // update to actual deployed URL
  icons: ['https://avatars.githubusercontent.com/u/179229932'] // Reown org avatar — OK
}

Note: This same pattern exists across other Next.js examples in the repo (next-ethers-app-router, etc.), so a follow-up cleanup across all examples would be beneficial.

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.

1 participant