Skip to content

DOC-13344 fix bad links #890

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: release/3.2
Choose a base branch
from

Conversation

osfameron
Copy link
Contributor

@osfameron osfameron commented Jul 8, 2025

Preview Build docs-sync-gateway-DOC-13344-unresolved-directive

Note

This is a preview build1 of the Couchbase documentation site.
It is not intended for production use and may contain incomplete or experimental content.

Preview URL:
https://preview.docs-test.couchbase.com/docs-sync-gateway-DOC-13344-unresolved-directive

You will need the Docs Team credentials on Confluence.

Changed pages


unresolved links due to e.g. partial file having been renamed, or missing page attribute.

Preview URL:
https://preview.docs-test.couchbase.com/docs-sync-gateway-DOC-13344-unresolved-directive

You will need the Docs Team credentials on Confluence.

Footnotes

  1. This preview was built from the branch DOC-13344-unresolved-directive of the repository docs-sync-gateway by osfameron at workflow_dispatch.
    See the preview tree.

unresolved links due to e.g. partial file having been renamed, or missing page attribute.
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes unresolved include directives in AsciiDoc pages and adds a missing docs-couchbase-lite source.

  • Adds the docs-couchbase-lite branch entry in preview/HEAD.yml
  • Updates the include in authentication-users.adoc to point at the renamed partial
  • Renames the include in incpg-icr-monitoring.adoc to the new define_stats_item_names partial

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
preview/HEAD.yml Added docs-couchbase-lite source under sources:
modules/ROOT/pages/authentication-users.adoc Changed include to partial$blocklinks-cbl.adoc[]
modules/ROOT/pages/_partials/incpg-icr-monitoring.adoc Renamed include to partial$_define_stats_item_names.adoc[]
Comments suppressed due to low confidence (2)

modules/ROOT/pages/_partials/incpg-icr-monitoring.adoc:2

  • [nitpick] Verify that the partial filename (define_stats_item_names.adoc) matches the actual file and follows the project’s naming conventions (mixing hyphens and underscores may be confusing).
include::partial$_define_stats_item_names.adoc[]

modules/ROOT/pages/authentication-users.adoc:135

  • The include directive is missing an underscore after the $. It should be include::partial$_blocklinks-cbl.adoc[] to correctly resolve the partial.
include::partial$blocklinks-cbl.adoc[]

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.

3 participants