Skip to content
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

DOC-11757: remove page #838

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

Conversation

ElliotFrancisHunter
Copy link
Collaborator

https://issues.couchbase.com/browse/DOC-11757

This is a fix for the issue below:
A user stumbled across this page https://docs.couchbase.com/couchbase-lite/current/c/p2psync-websocket-using-passive.html which really simply shouldn't exist. This feature is unimplemented in C
(hence all the blank code snippets)

Page had been removed from the nav but not from the repo so users could get there through searching/url hacking.

@ElliotFrancisHunter ElliotFrancisHunter added bug Something isn't working docs labels Jan 5, 2024
@ElliotFrancisHunter ElliotFrancisHunter requested review from osfameron, bfavini and a team January 5, 2024 16:45
@ElliotFrancisHunter ElliotFrancisHunter self-assigned this Jan 5, 2024
Copy link
Contributor

@RichardSmedley RichardSmedley left a comment

Choose a reason for hiding this comment

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

It might be a good idea to add the page-alias for this page, and the page alias contained within this page, to another page - such as https://github.com/couchbase/docs-couchbase-lite/blob/release/3.1/modules/c/pages/p2psync-websocket.adoc
...which will stop anyone who has bookmarked this page, waiting for content to appear, from getting a 404.

@osfameron
Copy link
Collaborator

Agree with @RichardSmedley, did I take you through :page-aliases: yet @ElliotFrancisHunter ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants