Skip to content

Conversation

@Provizor071
Copy link

Converted absolute links to relative MDX paths in these files:

  • docs/base-account/cdp.mdx
  • docs/base-account/framework-integrations/nextjs-with-dynamic.mdx
  • docs/base-account/sub-accounts.mdx (2 links)

Reason: absolute /base-account/... links can break in nested deployments. Relative MDX paths resolve consistently.

Replaced absolute site-root links with working relative MDX links:

- /base-account/framework-integrations/wagmi/setup
→ ./framework-integrations/wagmi/setup.mdx

- /base-account/framework-integrations/wagmi/sign-in-with-base
→ ./framework-integrations/wagmi/sign-in-with-base.mdx

This prevents broken navigation in nested routes and ensures correct resolution in different build environments.
@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Oct 21, 2025

🟡 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

@kultama
Copy link

kultama commented Oct 21, 2025

nice

Corrected relative MDX paths for wagmi links:
- ./framework-integrations/wagmi/setup.mdx → ./wagmi/setup.mdx
- ./framework-integrations/wagmi/setup.mdx → ./setup.mdx in sub-accounts
- fixed sign-in-with-base link
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.

3 participants