Skip to content

DOC-17064: Replace issue filing/search links#23300

Merged
ebembi-crdb merged 6 commits into
mainfrom
unlink-issue-filing-links
May 14, 2026
Merged

DOC-17064: Replace issue filing/search links#23300
ebembi-crdb merged 6 commits into
mainfrom
unlink-issue-filing-links

Conversation

@ebembi-crdb
Copy link
Copy Markdown
Contributor

Summary

  • Unlinks 54 pages that reference github.com/cockroachdb/cockroach/issues/new or issues?q= URLs
  • Removes hyperlinks directing users to file or search issues in the cockroach repo
  • Preserves descriptive link text without the hyperlink

This is PR 10 of 10 splitting #23256 into reviewable chunks.

Jira: DOC-17064

Test plan

  • Verify unlinked text still reads naturally (e.g., "open an issue" without link)
  • Run local Jekyll build and spot-check affected pages
  • Confirm no remaining cockroachdb/cockroach/issues/new or issues?q= links in affected files

🤖 Generated with Claude Code

Unlink 54 pages that reference github.com/cockroachdb/cockroach/issues/new
or issues?q= URLs. These links directed users to file or search issues
in the cockroach repo; the link text is preserved without the hyperlink.

Part of DOC-17064.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 12, 2026

Files changed:

@netlify
Copy link
Copy Markdown

netlify Bot commented May 12, 2026

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit d6189f9
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-api-docs/deploys/6a05d99ea2f2310008d61c7f

@netlify
Copy link
Copy Markdown

netlify Bot commented May 12, 2026

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit d6189f9
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-interactivetutorials-docs/deploys/6a05d99eb729d400083070b0

@netlify
Copy link
Copy Markdown

netlify Bot commented May 12, 2026

Netlify Preview

Name Link
🔨 Latest commit d6189f9
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-docs/deploys/6a05d99e047b3f00090f052c
😎 Deploy Preview https://deploy-preview-23300--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.

Copy link
Copy Markdown
Contributor

@rmloveland rmloveland left a comment

Choose a reason for hiding this comment

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

@ebembi-crdb please update to remove not just the link targets but also the text that refers to issues like cockroachdb/cockroach#... etc

also things that say "file an issue" should either be deleted or updated to say "contact Support" with a link to our support docs

my comments on v23.1 apply pretty much the same to all versions in the PR AFAICT

Comment thread src/current/v23.1/example-apps.md Outdated
Comment thread src/current/v23.1/file-an-issue.md Outdated
Comment thread src/current/v23.1/intellij-idea.md Outdated
Comment thread src/current/v23.1/postgresql-compatibility.md Outdated
Comment thread src/current/v23.1/postgresql-compatibility.md Outdated
Comment thread src/current/v23.2/example-apps.md Outdated
Comment thread src/current/v24.2/postgresql-compatibility.md Outdated
Comment thread src/current/v24.2/postgresql-compatibility.md Outdated
Comment thread src/current/v26.2/active-session-history.md Outdated
Comment thread src/current/v26.2/file-an-issue.md Outdated
ebembi-crdb and others added 2 commits May 13, 2026 16:39
…e file-an-issue.md

Address reviewer feedback on PR #23300 by substantively rewriting or
removing all GitHub issue-filing references rather than merely unlinking:

- Replace "open an issue" with "contact Support" link in example-apps.md
  and install-client-drivers.md across all versions
- Rewrite intellij-idea.md to direct users to Support instead of GitHub
- Remove cockroachdb/cockroach#NNNNN issue refs, GitHub label references,
  and tracking issue lines from postgresql-compatibility.md
- Replace "file an issue" with "contact Support" in active-session-history.md
- Delete file-an-issue.md pages across all 11 versions
- Remove "File an Issue" sidebar entries from troubleshooting.json
- Update ~120 files referencing file-an-issue.md to link to
  support-resources.md instead

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Resolve conflicts from PR #23225 (page deletion batch 6) which also
removed file-an-issue.md in v26.1+. Keep our "contact Support" links
in .md files; use main's formatting for sidebar JSON files.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 13, 2026

Cross-Version Link Check Passed

All cross-version link issues have been resolved. Good job!

Replace hardcoded v24.2 image paths with {{ page.version.version }}
in v25.1, v25.2, and v25.3 to fix CI cross-version link check.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@rmloveland rmloveland left a comment

Choose a reason for hiding this comment

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

LGTM once you get the build working - see my comment below re: the image link syntax needs to be hardcoded to match the page version

Comment thread src/current/v25.1/query-behavior-troubleshooting.md Outdated
Comment thread src/current/v25.2/query-behavior-troubleshooting.md Outdated
Image paths in img src attributes must use hardcoded version strings,
not {{ page.version.version }}, to avoid build failures. Set v25.1,
v25.2, and v25.3 image paths to their respective versions.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rmloveland
Copy link
Copy Markdown
Contributor

@ebembi-crdb this is good to merge as far as i'm concerned! thank you for doing this!

@ebembi-crdb ebembi-crdb enabled auto-merge May 14, 2026 14:18
@ebembi-crdb ebembi-crdb merged commit 382f553 into main May 14, 2026
7 checks passed
@ebembi-crdb ebembi-crdb deleted the unlink-issue-filing-links branch May 14, 2026 14:25
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