Skip to content
This repository was archived by the owner on Jul 30, 2025. It is now read-only.

Conversation

@gregnazario
Copy link
Collaborator

Description

Checklist

  • If any existing pages were renamed or removed:
    • Were redirects added to next.config.mjs?
    • Did you update any relative links that pointed to the renamed / removed pages?
  • Do all Lints pass?
    • Have you ran pnpm fmt?
    • Have you ran pnpm lint?

Copilot AI review requested due to automatic review settings June 30, 2025 04:29
@gregnazario gregnazario requested a review from a team as a code owner June 30, 2025 04:29
@vercel
Copy link

vercel bot commented Jun 30, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
developer-docs-nextra ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 30, 2025 4:36am

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adds OKX Connect instructions to the Wallet Adapter documentation and restructures the content into separate sections.

  • Introduce a list of two wallet adapter standards (Aptos and OKX).
  • Reorganize headings to distinguish Aptos Wallet Adapter and OKX Connect.
  • Add an "Other Resources" section at the bottom.
Comments suppressed due to low confidence (3)

apps/nextra/pages/en/build/sdks/wallet-adapter.mdx:41

  • Replace the double space after the period and consider pluralizing "OKX wallet" to "OKX wallets" for clarity: "...to connect to OKX wallets and other wallets that support the OKX Connect standard. You can find".
wallet and other wallets that support the OKX Connect standard.  You can find

apps/nextra/pages/en/build/sdks/wallet-adapter.mdx:47

- [Dapp Builder Guide](./wallet-adapter/dapp.mdx)

apps/nextra/pages/en/build/sdks/wallet-adapter.mdx:45

  • [nitpick] Consider adding a bullet for the OKX Connect documentation link under this section to improve discoverability.
## Other Resources

@gregnazario gregnazario merged commit 4043a5e into main Jun 30, 2025
4 of 5 checks passed
@gregnazario gregnazario deleted the add-okx-connect branch June 30, 2025 04:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants