[PR #338] [MERGED] Clean up endpoints #506

Closed
opened 2026-02-16 01:15:11 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langserve/pull/338
Author: @eyurtsev
Created: 12/20/2023
Status: Merged
Merged: 12/20/2023
Merged by: @eyurtsev

Base: mainHead: eugene/cleanup_endpoints


📝 Commits (1)

📊 Changes

3 files changed (+6 additions, -23 deletions)

View changed files

📝 langserve/api_handler.py (+4 -4)
📝 langserve/server.py (+2 -18)
📝 tests/unit_tests/test_server_client.py (+0 -1)

📄 Description

Remove feedback related endpoints that take a config hash. The feedback endpoints only care about the run_id and the feedback, so a config hash should not be needed.

Also removing the get version of checking if feedback is enabled. I'll transition both the UI and the backend in a bit.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/langserve/pull/338 **Author:** [@eyurtsev](https://github.com/eyurtsev) **Created:** 12/20/2023 **Status:** ✅ Merged **Merged:** 12/20/2023 **Merged by:** [@eyurtsev](https://github.com/eyurtsev) **Base:** `main` ← **Head:** `eugene/cleanup_endpoints` --- ### 📝 Commits (1) - [`642e7a2`](https://github.com/langchain-ai/langserve/commit/642e7a26a3af4a34fce1ba01f69b7b59f61a3df2) x ### 📊 Changes **3 files changed** (+6 additions, -23 deletions) <details> <summary>View changed files</summary> 📝 `langserve/api_handler.py` (+4 -4) 📝 `langserve/server.py` (+2 -18) 📝 `tests/unit_tests/test_server_client.py` (+0 -1) </details> ### 📄 Description Remove feedback related endpoints that take a config hash. The feedback endpoints only care about the run_id and the feedback, so a config hash should not be needed. Also removing the `get` version of checking if feedback is enabled. I'll transition both the UI and the backend in a bit. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 01:15:11 -05:00
yindo closed this issue 2026-02-16 01:15:11 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/langserve#506