Skip to content

docs: fix 3 broken links in README and cloud-cli#1026

Closed
sanjibani wants to merge 1 commit into
basicmachines-co:mainfrom
sanjibani:fix/broken-docs-links
Closed

docs: fix 3 broken links in README and cloud-cli#1026
sanjibani wants to merge 1 commit into
basicmachines-co:mainfrom
sanjibani:fix/broken-docs-links

Conversation

@sanjibani

Copy link
Copy Markdown

The basicmemory.com docs site restructured: /getting-started/note-formatting and /guides/cli-reference both 404. Canonical replacements are /concepts/knowledge-format and /reference/cli-reference (the old /guides prefix was renamed to /reference). The basicmemory.com/subscribe landing page was retired; replaced with /pricing.

Verified via curl GET: all three new URLs return 200.

2 files, +3/-3.

The basicmemory.com docs site restructured: /getting-started/note-formatting and /guides/cli-reference both 404, replaced with the canonical /concepts/knowledge-format and /reference/cli-reference paths (the old /guides prefix was renamed to /reference). The basicmemory.com/subscribe landing page was retired and now redirects to /pricing.

Verified via curl: GET on the new URLs returns 200.

Signed-off-by: sanjibani <18418553+sanjibani@users.noreply.github.com>
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@phernandez

Copy link
Copy Markdown
Member

@sanjibani thanks for the submission. Do you mind signing the cla?

@sanjibani

Copy link
Copy Markdown
Author

recheck

@phernandez phernandez added the cla needed The contributor needs to sign the CLA label Jul 15, 2026 — with ChatGPT Codex Connector

Copy link
Copy Markdown
Member

Thanks for the contribution. The CLA check is still pending, so we cannot merge this PR yet.

Please complete or re-authorize the contributor agreement so the check turns green. Once it is green, maintainers can take over any remaining implementation changes needed to get this merged.

@phernandez

Copy link
Copy Markdown
Member

Thank you @sanjibani for finding and reporting these valid broken links. Because the CLA remained incomplete, we independently reimplemented the fixes in maintainer PR #1083 (without copying the contributor commit), including the related CLI fallback URL. #1083 has now merged, so I am closing this PR as superseded.

@phernandez phernandez closed this Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla needed The contributor needs to sign the CLA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants