Skip to content

Conversation

@ccurme
Copy link
Contributor

@ccurme ccurme commented Oct 28, 2025

  1. We refer to Chat Completions as a legacy API— as far as I know this is not the case (let me know if OpenAI documents that somewhere)
  2. We link to the legacy Completions API, which is not the same as Chat Completions

Copilot AI review requested due to automatic review settings October 28, 2025 20:00
@ccurme ccurme requested a review from mdrxy as a code owner October 28, 2025 20:00
@github-actions github-actions bot added langchain For docs changes to LangChain python For content related to the Python version of LangChain projects oss labels Oct 28, 2025
@github-actions
Copy link
Contributor

Mintlify preview ID generated: preview-ccchat-1761681688-11a62bf

Copy link
Contributor

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 updates documentation to correct the URL and description for OpenAI's Chat Completions API, removing the outdated "(legacy)" designation and pointing to the current API reference.

  • Updated OpenAI Chat Completions API link from the legacy /docs/guides/completions to the current /docs/api-reference/chat
  • Removed "(legacy)" designation from the Chat Completions API references

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/oss/python/integrations/chat/openai.mdx Updated Chat Completions API link and removed legacy designation in the compatibility note
src/oss/python/integrations/chat/index.mdx Updated Chat Completions API link and removed legacy designation in the Chat Completions API section

</Tip>

<Note>
**Chat Completions API compatiblity**
Copy link

Copilot AI Oct 28, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'compatiblity' to 'compatibility'.

Copilot uses AI. Check for mistakes.
</Tip>

<Note>
**Chat Completions API compatiblity**
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
**Chat Completions API compatiblity**
**Chat Completions API compatibility**

@github-actions
Copy link
Contributor

Mintlify preview ID generated: preview-ccchat-1761681870-258983f

@ccurme ccurme merged commit a5f881a into main Oct 28, 2025
11 checks passed
@ccurme ccurme deleted the cc/chat_completions_links branch October 28, 2025 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

langchain For docs changes to LangChain oss python For content related to the Python version of LangChain projects

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants