Skip to content

[Feature] UI - Spend Logs: Settings Modal - #19918

Merged
yuneng-jiang merged 4 commits into
mainfrom
litellm_ui_spend_logs_store
Jan 28, 2026
Merged

[Feature] UI - Spend Logs: Settings Modal#19918
yuneng-jiang merged 4 commits into
mainfrom
litellm_ui_spend_logs_store

Conversation

@yuneng-jiang

Copy link
Copy Markdown
Contributor

Relevant issues

This PR implements the Spend Logs Settings Modal. Currently this only shows the store_prompt_in_spend_logs and the retention period. This is meant to allow UI users to configure spend logs to start storing the request/response pair without restart of the instance

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • My PR passes all unit tests on make test-unit
  • My PR's scope is as isolated as possible, it only solves 1 specific problem

CI (LiteLLM team)

CI status guideline:

  • 50-55 passing tests: main is stable with minor issues.
  • 45-49 passing tests: acceptable but needs attention
  • <= 40 passing tests: unstable; be careful with your merges and assess the risk.
  • Branch creation CI run
    Link:

  • CI run for the last commit
    Link:

  • Merge / cherry-pick CI run
    Links:

Type

🆕 New Feature
✅ Test

Changes

image image

@vercel

vercel Bot commented Jan 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
litellm Error Error Jan 28, 2026 9:44pm

Request Review

@yuneng-jiang
yuneng-jiang merged commit 054918e into main Jan 28, 2026
50 of 63 checks passed
@ishaan-berri
ishaan-berri deleted the litellm_ui_spend_logs_store branch March 26, 2026 22:30
fzowl pushed a commit to fzowl/litellm that referenced this pull request Jun 24, 2026
…tore

[Feature] UI - Spend Logs: Settings Modal
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.

1 participant