Skip to content

[Trace] Broken links: fix dead links in the docs site #168

Description

@nicolasiscoding

Change type

Standard — low-risk, pre-approved

Risk / impact

Low

Security impact assessed?

No — no security impact

Details — description & full context

Correction: The broken-link checker that produced this CR resolved Docusaurus
file-relative links (e.g. ./setting-up-automation.md, ../../TurboSign/Webhooks.md)
against the site root instead of against the linking file's own directory. That made it
misreport a large number of working links as broken, so the original "56 broken links"
count was inflated. Verified on the live site: docs/Integrations/Wrike/index.md's
./setting-up-automation.md correctly renders as
https://docs.turbodocx.com/docs/Integrations/Wrike/setting-up-automation (200).

The linked PR was reworked to (1) revert the 25 files where the checker's report was a
false positive back to their original relative links, and (2) keep only the genuine
fixes:

  • Webhook Integration / Webhook Configuration (in API Templates.md and
    ai-variable-generation.md) pointed at /docs/Webhooks/webhook-configuration, which
    does not exist. Repointed to the real webhooks page: /docs/TurboSign/Webhooks.
  • API Authentication (same two files) had been pointed at
    /docs/API/Deliverable%20API by the first pass of this CR. Its original target,
    /docs/API/turbodocx-api-documentation, actually resolves (confirmed 200 live), so it
    was reverted to that.
  • Bulk Document Generation / Bulk Processing, Template Version Management, and
    Template Management Guide pointed at pages that don't exist anywhere in the docs
    site (/docs/Templates/*). No existing page covers this content, so these bullets
    were removed outright rather than replaced with a "(coming soon)" placeholder, which
    would publicly promise a feature/doc that may never ship.
  • Integration Examples pointed at /docs/Integrations. That category page is
    configured as a Docusaurus generated-index, but it actually 404s on the live site,
    so this is a genuine (if minor) site issue outside this CR's scope; the dead bullet
    was removed rather than left linking to a 404.

Net result: 3 files changed (down from 28), removing 2 lines and fixing 2 bullets to
point at real pages. git diff origin/develop...HEAD now shows only these real fixes.

Component / area

Documentation — internal cross-linking

Testing & validation

  • Verified every link change against the file tree in the worktree (file exists at
    the resolved relative/absolute path)
  • Spot-checked ambiguous cases against the live site with curl -o /dev/null -w '%{http_code}' (e.g. /docs/TurboSign/Webhooks → 200, /docs/Integrations → 404,
    /docs/API/turbodocx-api-documentation → 200)
  • Broken-link checker re-run against the merged result (recommend fixing the
    checker's relative-link resolution before trusting its next report)

Rollback plan

Revert the merge commit to restore the previous (checker-driven) link state.

Breaking change for consumers?

No

Reviewer / approver

@me

No activity

Activity on this issue will appear here.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

changeSOC 2 change management recorddocumentationImprovements or additions to documentation

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions