Skip to content

docs: fix relative link depth in evals reference and vector stores pages - #6496

Open
simpleqt wants to merge 1 commit into
ogx-ai:mainfrom
simpleqt:docs/fix-relative-link-depth
Open

simpleqt wants to merge 1 commit into
ogx-ai:mainfrom
simpleqt:docs/fix-relative-link-depth

Conversation

@simpleqt

@simpleqt simpleqt commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Three relative links resolved to nonexistent paths:

  • docs/docs/references/evals_reference/index.mdx: [Core Concepts](../concepts/) and [OGX Playground](../building_applications/playground) resolved one level short of docs/docs/concepts / docs/docs/building_applications; added the missing ../.
  • docs/docs/concepts/file_operations_vector_stores.mdx: [Files API documentation](../../providers/files/) escaped the docs root (docs/providers/ does not exist); the Files API docs live at docs/docs/providers/files/; changed to ../providers/files/.

- evals_reference/index.mdx linked ../concepts/ and
  ../building_applications/playground, resolving one level short of
  docs/docs/concepts and docs/docs/building_applications
- file_operations_vector_stores.mdx linked ../../providers/files/,
  escaping the docs root; the Files API docs live at
  docs/docs/providers/files/

Signed-off-by: simpleqt <89645338+simpleqt@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings September 4, 2026 18:43

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@mattf mattf left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

those links were wrong, but the proposed ones need review. please double check.

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