[PR #24770] refactor: remove unused GET /evaluations/feedbacks/user endpoint #115141

Open
opened 2026-05-18 16:06:02 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/open-webui/open-webui/pull/24770
Author: @Classic298
Created: 5/15/2026
Status: 🔄 Open

Base: devHead: remove-unused-feedbacks-user-endpoint


📝 Commits (1)

  • ccbe597 refactor: remove unused GET /evaluations/feedbacks/user endpoint

📊 Changes

2 files changed (+0 additions, -12 deletions)

View changed files

📝 backend/open_webui/models/feedbacks.py (+0 -5)
📝 backend/open_webui/routers/evaluations.py (+0 -7)

📄 Description

This endpoint loaded a user's entire feedback table into memory in one response (flagged as a High OOM risk in open-webui#22206), but it has no callers anywhere: nothing in the frontend, backend, tests, or docs invokes it. Rather than add streaming/pagination to dead code, remove it.

Also drops the now-orphaned Feedbacks.get_feedbacks_by_user_id data layer method (its only caller was this endpoint) and the now-unused FeedbackUserResponse import in the evaluations router.

Ref: open-webui#22206

Contributor License Agreement

Note

Deleting the CLA section will lead to immediate closure of your PR and it will not be merged in.


🔄 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/open-webui/open-webui/pull/24770 **Author:** [@Classic298](https://github.com/Classic298) **Created:** 5/15/2026 **Status:** 🔄 Open **Base:** `dev` ← **Head:** `remove-unused-feedbacks-user-endpoint` --- ### 📝 Commits (1) - [`ccbe597`](https://github.com/open-webui/open-webui/commit/ccbe59709c34c4fc4a4e756864dc45cb119cfba4) refactor: remove unused GET /evaluations/feedbacks/user endpoint ### 📊 Changes **2 files changed** (+0 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `backend/open_webui/models/feedbacks.py` (+0 -5) 📝 `backend/open_webui/routers/evaluations.py` (+0 -7) </details> ### 📄 Description This endpoint loaded a user's entire feedback table into memory in one response (flagged as a High OOM risk in open-webui#22206), but it has no callers anywhere: nothing in the frontend, backend, tests, or docs invokes it. Rather than add streaming/pagination to dead code, remove it. Also drops the now-orphaned Feedbacks.get_feedbacks_by_user_id data layer method (its only caller was this endpoint) and the now-unused FeedbackUserResponse import in the evaluations router. Ref: open-webui#22206 ### Contributor License Agreement <!-- 🚨 DO NOT DELETE THE TEXT BELOW 🚨 Keep the "Contributor License Agreement" confirmation text intact. Deleting it will trigger the CLA-Bot to INVALIDATE your PR. Your PR will NOT be reviewed or merged until you check the box below confirming that you have read and agree to the terms of the CLA. --> - [x] By submitting this pull request, I confirm that I have read and fully agree to the [Contributor License Agreement (CLA)](https://github.com/open-webui/open-webui/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT), and I am providing my contributions under its terms. > [!NOTE] > Deleting the CLA section will lead to immediate closure of your PR and it will not be merged in. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-05-18 16:06:02 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/open-webui#115141