Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Jan 27, 2026

What

Fixes broken links on the Core Concepts documentation page reported in Slack. The links under "Sync Schedule", "Destination Namespace", and "Delivery Method" sections were returning 404 errors.

How

The Core Concepts page is rendered from a readme.md file in a directory, making it an index page. With trailingSlash: false in vercel.json, the URL becomes /platform/using-airbyte/core-concepts (no trailing slash). When browsers resolve relative links like sync-schedules from this URL, they treat core-concepts as a file rather than a directory, causing the link to resolve to /platform/using-airbyte/sync-schedules instead of /platform/using-airbyte/core-concepts/sync-schedules.

Fixed by converting relative links to absolute paths:

  • sync-schedules/platform/using-airbyte/core-concepts/sync-schedules
  • namespaces/platform/using-airbyte/core-concepts/namespaces
  • ../delivery-methods/platform/using-airbyte/delivery-methods

Applied to current docs and all versioned docs (1.6, 1.7, 1.8, 2.0).

Review guide

  1. docs/platform/using-airbyte/core-concepts/readme.md - main source file
  2. Versioned docs in docusaurus/platform_versioned_docs/ - identical changes

Key verification: Confirm the absolute paths resolve correctly in both current and versioned documentation contexts.

User Impact

Users will no longer encounter 404 errors when clicking the Sync Schedules, Namespaces, or Delivery Methods links on the Core Concepts page.

Can this PR be safely reverted and rolled back?

  • YES 💚

Requested by: ian.alton@airbyte.io

Link to Devin run: https://app.devin.ai/sessions/c4eb2e4e161443a2a9a7293d74d4adb4

Important

Auto-merge enabled.

This PR is set to merge automatically when all requirements are met.

The Core Concepts page uses relative links that resolve incorrectly
due to the trailingSlash: false setting in vercel.json. When the URL
is /platform/using-airbyte/core-concepts (no trailing slash), the
browser treats core-concepts as a file, not a directory, causing
relative links to resolve to the wrong path.

Fixed by converting relative links to absolute paths:
- sync-schedules -> /platform/using-airbyte/core-concepts/sync-schedules
- namespaces -> /platform/using-airbyte/core-concepts/namespaces
- ../delivery-methods -> /platform/using-airbyte/delivery-methods

Applied to current docs and all versioned docs (1.6, 1.7, 1.8, 2.0).

Co-Authored-By: ian.alton@airbyte.io <ian.alton@airbyte.io>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions
Copy link
Contributor

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

💡 Show Tips and Tricks

PR Slash Commands

Airbyte Maintainers (that's you!) can execute the following slash commands on your PR:

  • 🛠️ Quick Fixes
    • /format-fix - Fixes most formatting issues.
    • /bump-version - Bumps connector versions, scraping changelog description from the PR title.
  • ❇️ AI Testing and Review:
    • /ai-docs-review - Provides AI-powered documentation recommendations for PRs with connector changes.
    • /ai-prove-fix - Runs prerelease readiness checks, including testing against customer connections.
    • /ai-canary-prerelease - Rolls out prerelease to 5-10 connections for canary testing.
    • /ai-review - AI-powered PR review for connector safety and quality gates.
  • 🚀 Connector Releases:
    • /publish-connectors-prerelease - Publishes pre-release connector builds (tagged as {version}-preview.{git-sha}) for all modified connectors in the PR.
    • /bump-progressive-rollout-version - Bumps connector version with an RC suffix (2.16.10-rc.1) for progressive rollouts (enableProgressiveRollout: true).
      • Example: /bump-progressive-rollout-version changelog="Add new feature for progressive rollout"
  • ☕️ JVM connectors:
    • /update-connector-cdk-version connector=<CONNECTOR_NAME> - Updates the specified connector to the latest CDK version.
      Example: /update-connector-cdk-version connector=destination-bigquery
    • /bump-bulk-cdk-version bump=patch changelog='foo' - Bump the Bulk CDK's version. bump can be major/minor/patch.
  • 🐍 Python connectors:
    • /poe connector source-example lock - Run the Poe lock task on the source-example connector, committing the results back to the branch.
    • /poe source example lock - Alias for /poe connector source-example lock.
    • /poe source example use-cdk-branch my/branch - Pin the source-example CDK reference to the branch name specified.
    • /poe source example use-cdk-latest - Update the source-example CDK dependency to the latest available version.
  • ⚙️ Admin commands:
    • /force-merge reason="<REASON>" - Force merges the PR using admin privileges, bypassing CI checks. Requires a reason.
      Example: /force-merge reason="CI is flaky, tests pass locally"
📚 Show Repo Guidance

Helpful Resources

📝 Edit this welcome message.

@github-actions
Copy link
Contributor

Deploy preview for airbyte-docs ready!

✅ Preview
https://airbyte-docs-6hb9l0kbx-airbyte-growth.vercel.app

Built with commit 030a421.
This pull request is being automatically deployed with vercel-action

@ian-at-airbyte ian-at-airbyte marked this pull request as ready for review January 28, 2026 00:15
@ian-at-airbyte ian-at-airbyte enabled auto-merge (squash) January 28, 2026 00:16
@ian-at-airbyte ian-at-airbyte merged commit 8a38043 into master Jan 28, 2026
54 checks passed
@ian-at-airbyte ian-at-airbyte deleted the devin/1769554707-fix-core-concepts-broken-links branch January 28, 2026 00:18
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.

1 participant