Skip to content

fix(restapi): resolve broken markdown link breaking prod build#190

Merged
ecunado merged 1 commit into
devfrom
fix/restapi-2.28-broken-link
Jun 30, 2026
Merged

fix(restapi): resolve broken markdown link breaking prod build#190
ecunado merged 1 commit into
devfrom
fix/restapi-2.28-broken-link

Conversation

@ecunado

@ecunado ecunado commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Problem

The production deploy (devmain) failed at yarn deploy:

Markdown link with URL `restapi/restobjects.md#messageReasonCode` in
"restapi_versioned_docs/version-REST API 2.28.0/restendpoints.md" (1386:140) couldn't be resolved.

Failed run

Cause

Within a docs plugin, links must be relative to the plugin itself. Line 1386 used the restapi/ prefix, which points outside the plugin and fails to resolve. With onBrokenMarkdownLinks: 'throw' this aborts the entire build. Introduced in 03a3924 restApi v2.28.

Fix

- See [allowed values](restapi/restobjects.md#messageReasonCode)|
+ See [allowed values](restobjects.md#messageReasonCode)|

Applied to both the live file and the versioned (2.28.0) copy.

Verification

Local yarn build (with lockfile deps) → [SUCCESS] Generated static files. The remaining broken anchors are pre-existing warnings only and do not block the deploy.

The link in restendpoints.md (line 1386) used the `restapi/` plugin
prefix, which points outside the restapi docs plugin and fails to
resolve. With `onBrokenMarkdownLinks: 'throw'` this aborted the
production build/deploy. Make it relative to the plugin like every
other link in the file.

Fixed in both the live and versioned (2.28.0) copies.
@ecunado ecunado force-pushed the fix/restapi-2.28-broken-link branch from 7d9a1af to 1e61a78 Compare June 30, 2026 09:07
@ecunado ecunado merged commit 3cfe719 into dev Jun 30, 2026
2 checks passed
@ecunado ecunado deleted the fix/restapi-2.28-broken-link branch June 30, 2026 09:32
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