Skip to content

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

Open
ygd58 wants to merge 1 commit intobase:masterfrom
ygd58:fix/remove-mdx-extension-signInWithEthereum
Open

fix(docs): remove .mdx extension from signInWithEthereum link in wallet_connect reference#1312
ygd58 wants to merge 1 commit intobase:masterfrom
ygd58:fix/remove-mdx-extension-signInWithEthereum

Conversation

@ygd58
Copy link
Copy Markdown

@ygd58 ygd58 commented Apr 14, 2026

Was looking at the wallet connect reference page and noticed the signInWithEthereum capability link was broken. The link had a .mdx extension at the end which Mintlify does not handle as a valid path so it returns 404. Removed the extension and the link works fine now. The file itself exists at docs/base account/reference/core/capabilities/signInWithEthereum.mdx but the URL should not include the extension. Single line change on line 193 of wallet connect.mdx.

@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented Apr 14, 2026

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
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 0
Sum 1

@ygd58 ygd58 force-pushed the fix/remove-mdx-extension-signInWithEthereum branch from e67e5db to 7280f25 Compare April 18, 2026 06:40
@ygd58
Copy link
Copy Markdown
Author

ygd58 commented Apr 18, 2026

hey @roethke @soheimam can you take a look when you get a chance? small fix

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