-
Notifications
You must be signed in to change notification settings - Fork 39
Release/3.3 module refactoring #912
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
Conversation
also mv partials to ROOT/partials
release-notes, logging, sync-function
There was a problem hiding this 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 pull request implements a comprehensive module refactoring for the Sync Gateway documentation site to support version 3.3. The main purpose is to restructure the content organization by moving pages from the ROOT module to specialized modules while updating all cross-references accordingly.
Key changes include:
- Moving documentation pages from ROOT module to specialized modules (sync, deploy, manage, security, etc.)
- Updating all internal cross-references to use explicit module prefixes (e.g.,
ROOT:,sync:,deploy:) - Adding page aliases to maintain backward compatibility with old URLs
- Creating new module-specific API reference pages for REST APIs
Reviewed Changes
Copilot reviewed 155 out of 220 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| preview/release/3.3-module-refactoring.yml | Configuration file for the 3.3 module refactoring preview |
| modules/use-kubernetes/pages/deploy-cluster-to-kubernetes.adoc | Updates cross-references to use ROOT module prefix for partials and aliases |
| modules/sync/pages/*.adoc | Multiple sync-related pages with updated cross-references and ROOT module prefixes |
| modules/start-here/pages/*.adoc | Getting started pages with updated cross-references to ROOT module |
| modules/server-compatibility/pages/*.adoc | Server compatibility pages with updated cross-references |
| modules/security/pages/*.adoc | Security documentation with updated cross-references and aliases |
| modules/rest-api/pages/*.adoc | New REST API reference pages and updated cross-references |
| modules/product-notes/pages/*.adoc | Product notes with updated cross-references |
| modules/manage/pages/*.adoc | Management documentation with updated cross-references |
| modules/deploy/pages/*.adoc | Deployment documentation with updated cross-references |
| modules/configuration/pages/*.adoc | Configuration documentation with updated cross-references |
| modules/access-control/pages/*.adoc | Access control documentation with updated cross-references |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| = Sync Function | ||
| :page-aliases: sync-function.adoc | ||
| :page-aliases: ROOT:sync-function.adoc | ||
| :page-aliases: ROOT:sync-function.adoc, ROOT:sync-function-overview.adoc, sync-function-overview.adoc |
Copilot
AI
Aug 20, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Duplicate alias 'ROOT:sync-function.adoc' appears in the page-aliases directive. The first alias should be sufficient.
| :page-aliases: ROOT:sync-function.adoc, ROOT:sync-function-overview.adoc, sync-function-overview.adoc | |
| :page-aliases: ROOT:sync-function-overview.adoc, sync-function-overview.adoc |
omits sgw 2.7 as that link to Star Channel is currently broken on prod also.
iamfortune
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Preview: https://preview.docs-test.couchbase.com/docs-sync-gateway-release/3.3-module-refactoring/