Skip to content

docs(base-account): drop .mdx extension from signInWithEthereum link#1333

Open
MukundaKatta wants to merge 1 commit intobase:masterfrom
MukundaKatta:codex/fix-wallet-connect-mdx-ext
Open

docs(base-account): drop .mdx extension from signInWithEthereum link#1333
MukundaKatta wants to merge 1 commit intobase:masterfrom
MukundaKatta:codex/fix-wallet-connect-mdx-ext

Conversation

@MukundaKatta
Copy link
Copy Markdown

Summary

`docs/base-account/reference/core/provider-rpc-methods/wallet_connect.mdx:193` links to `/base-account/reference/core/capabilities/signInWithEthereum.mdx`. Mintlify routes are published without the `.mdx` suffix, so the link 404s.

Removed the extension so the link resolves to `/base-account/reference/core/capabilities/signInWithEthereum`.

Closes #1311

Testing

Docs-only change. Verified:

  • docs.base.org/base-account/reference/core/capabilities/signInWithEthereum resolves;
  • the same-directory sibling `capabilities/signInWithEthereum.mdx` exists in-repo.

Mintlify routes are served without the .mdx extension, so the current
link /base-account/reference/core/capabilities/signInWithEthereum.mdx
returns 404. Removed the suffix to match all sibling links.

Closes base#1311
@cb-heimdall
Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/2
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

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.

fix(docs): remove .mdx extension from signInWithEthereum link in wallet_connect reference

2 participants