Skip to content
This repository was archived by the owner on Oct 6, 2026. It is now read-only.

Fix cross-version links in v23.2 documentation - #22276

Open
ebembi-crdb wants to merge 5 commits into
mainfrom
fix-cross-version-links-v23.2
Open

ebembi-crdb wants to merge 5 commits into
mainfrom
fix-cross-version-links-v23.2

Conversation

@ebembi-crdb

Copy link
Copy Markdown
Contributor

Summary

  • Fix cross-version link violations in v23.2 documentation
  • Replace hardcoded version references with {{ page.version.version }}

Changes

  • 3 files modified
  • 12 cross-version links fixed

Test Plan

  • Cross-version link linter passes for v23.2
  • Jekyll build completes successfully

Replace hardcoded version references with dynamic version variable.
@netlify

netlify Bot commented Jan 22, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit 19bae72
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-api-docs/deploys/697b63d78dcfea0008cb7386

@netlify

netlify Bot commented Jan 22, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit 19bae72
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-interactivetutorials-docs/deploys/697b63d77a7d59000836284b

@github-actions

github-actions Bot commented Jan 22, 2026 •

Copy link
Copy Markdown

Files changed:

@netlify

netlify Bot commented Jan 22, 2026 •

Copy link
Copy Markdown

✅ Netlify Preview

Name Link
🔨 Latest commit 19bae72
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-docs/deploys/697b63d71d32a50008884f41
😎 Deploy Preview https://deploy-preview-22276--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

ebembi-crdb added 4 commits January 27, 2026 16:35
Replace broken Liquid syntax with working path format:
- From: {{ 'images/vX.X/...' | relative_url }}
- To: /docs/images/{{ page.version.version }}/...
Change /docs/images/{{ page.version.version }}/... to proper Liquid syntax
{{ 'images/v23.2/...' | relative_url }} for correct URL generation.
Change nested {{ page.version.version }} to hardcoded v23.2 in image paths.
Liquid cannot evaluate nested {{ }} inside strings.

@rmloveland rmloveland left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

This branch has not been deployed

No deployments
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants