Skip to content

docs(client): fix links to merged payments and connectors sections - #367

Merged
wixysam merged 1 commit into
mainfrom
fix-service-role-payments-link
Nov 4, 2026
Merged

wixysam merged 1 commit into
mainfrom
fix-service-role-payments-link

Conversation

@wixysam

@wixysam wixysam commented Nov 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • asServiceRole.payments linked to ServiceRolePaymentsModule, which docs(payments): review the payments reference and merge the service role methods into it #364 merged into the payments page. The createClient and createClientFromRequest reference pages pointed at ../interfaces/service-role-payments, which no longer exists: 16 broken links on mintlify-docs main, counting the locale copies. It now links to the payments page's "Service role methods" section.
  • The user-level connectors entry had the same problem with UserConnectorsModule and rendered as plain text. It now links to the connectors page's "App user connectors" section.

Both use absolute markdown links, like the ones in actors.types.ts and ai-gateway.types.ts, because {@link} can't target a type that has no page of its own.

The post-processing only rewrites links to merged types on the page that takes them in (linkMergedTypesInPage), so the same thing can happen the next time a module is merged.

Test plan

  • Regenerate the reference and confirm both links resolve on the createClient and createClientFromRequest pages

🤖 Generated with Claude Code

The asServiceRole.payments entry linked to ServiceRolePaymentsModule, which
is now merged into the payments page, so createClient and
createClientFromRequest pointed at a page that no longer exists (16 broken
links across locales). Link to the "Service role methods" section instead.

The user-level connectors entry had the same problem with
UserConnectorsModule and rendered without a link. Link it to the
"App user connectors" section.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

🚀 Package Preview Available!


Install this PR's preview build with npm:

npm i @base44-preview/sdk@0.8.58-beta.0-pr.367.230725c

Prefer not to change any import paths? Install using npm alias so your code still imports @base44/sdk:

npm i "@base44/sdk@npm:@base44-preview/sdk@0.8.58-beta.0-pr.367.230725c"

Or add it to your package.json dependencies:

{
  "dependencies": {
    "@base44/sdk": "npm:@base44-preview/sdk@0.8.58-beta.0-pr.367.230725c"
  }
}

Preview published to npm registry — try new features instantly!

@wixysam
wixysam merged commit eca23ed into main Nov 4, 2026
6 of 9 checks passed
@wixysam
wixysam deleted the fix-service-role-payments-link branch November 4, 2026 08:08
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