Skip to content

Conversation

@mcavdar
Copy link
Contributor

@mcavdar mcavdar commented Oct 27, 2025

Overview

Replace ModelRequestHandler with Callable[[ModelRequest], ModelResponse] similar to #1123

Type of change

Type: [Update existing documentation /bug/ Remove outdated content ]

Related issues/PRs

  • Linear issue:
  • Slack thread:

Checklist

  • I have read the contributing guidelines
  • I have tested my changes locally using docs dev
  • All code examples have been tested and work correctly
  • I have used root relative paths for internal links
  • I have updated navigation in src/docs.json if needed
  • I have gotten approval from the relevant reviewers
  • (Internal team members only / optional) I have created a preview deployment using the Create Preview Branch workflow

Additional notes

@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 27, 2025
@mcavdar mcavdar changed the title Replace ModelRequestHandler with Callable[[ModelRequest], ModelResponse] docs: Replace ModelRequestHandler with Callable[[ModelRequest], ModelResponse] Oct 27, 2025
@mdrxy mdrxy changed the title docs: Replace ModelRequestHandler with Callable[[ModelRequest], ModelResponse] docs: Replace ModelRequestHandler with Callable[[ModelRequest], ModelResponse] Oct 27, 2025
@lnhsingh
Copy link
Collaborator

lnhsingh commented Nov 6, 2025

LGTM from docs perspective but waiting on @sydney-runkle to review technically

@sydney-runkle sydney-runkle enabled auto-merge (squash) November 6, 2025 21:59
@mdrxy mdrxy added the external User is not a member of langchain-ai label Nov 15, 2025
@lnhsingh lnhsingh disabled auto-merge November 18, 2025 21:45
@lnhsingh lnhsingh merged commit 2af2c1c into langchain-ai:main Nov 18, 2025
1 check passed
lnhsingh added a commit that referenced this pull request Nov 20, 2025
…delResponse]` (#1136)

## Overview
Replace ModelRequestHandler with Callable[[ModelRequest], ModelResponse]
similar to #1123

## Type of change

**Type:** [Update existing documentation /bug/ Remove outdated content ]

## Related issues/PRs
<!-- 
Link to related issues, feature PRs, or discussions (if applicable)

To automatically close an issue when this PR is merged, use closing
keywords:
- "closes #123" or "fixes #123" or "resolves #123"

For regular references without auto-closing, just use:
- "#123" or "See issue #123"

Examples:
- closes #456 (will auto-close issue #456 when PR is merged)
- See #789 for context (will reference but not auto-close issue #789)
-->
- GitHub issue:
- Feature PR: See #1123 for context

<!-- For LangChain employees, if applicable: -->
- Linear issue:
- Slack thread:

## Checklist
<!-- Put an 'x' in all boxes that apply -->
- [x] I have read the [contributing guidelines](README.md)
- [x] I have tested my changes locally using `docs dev`
- [x] All code examples have been tested and work correctly
- [x] I have used **root relative** paths for internal links
- [x] I have updated navigation in `src/docs.json` if needed
- I have gotten approval from the relevant reviewers
- (Internal team members only / optional) I have created a preview
deployment using the [Create Preview Branch
workflow](https://github.com/langchain-ai/docs/actions/workflows/create-preview-branch.yml)

## Additional notes
<!-- Any other information that would be helpful for reviewers -->

---------

Co-authored-by: Mason Daugherty <github@mdrxy.com>
Co-authored-by: Mason Daugherty <mason@langchain.dev>
Co-authored-by: Sydney Runkle <54324534+sydney-runkle@users.noreply.github.com>
Co-authored-by: Lauren Hirata Singh <lauren@langchain.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external User is not a member of langchain-ai 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.

5 participants