feat: hide budgets UI when ENABLE_LITELLM is off - #520
Draft
dylan-openhands wants to merge 1 commit into
Draft
dylan-openhands wants to merge 1 commit into
dylan-openhands wants to merge 1 commit into
Conversation
dylan-openhands
added this pull request to stack #522
September 25, 2026 00:14
This was referenced Sep 25, 2026
Coverage report
This PR does not seem to contain any modification to coverable code. |
dylan-openhands
force-pushed
the
enable-litellm-flag-frontend
branch
from
September 26, 2026 19:52
67b7e01 to
cc64a93
Compare
Budgets and Your Budget render a FeatureDisabledScreen and skip their fetches; their nav entries are hidden.
dylan-openhands
force-pushed
the
enable-litellm-flag-frontend
branch
from
September 26, 2026 21:45
cc64a93 to
6fccba9
Compare
6 tasks
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
HUMAN:
AGENT:
Why
With
ENABLE_LITELLM=falsethe budget screens have no backend to talk to. Middle of the stack #519 → #520 → #521.Summary
FeatureDisabledScreen("Please enable LiteLLM to use this feature") and skip their data fetches.How to Test
cd frontend && npx vitest run __tests__/components/features/budgetsENABLE_LITELLM=false, open Settings: Budgets / Your Budget nav items are gone, and visiting the routes directly shows the placeholder.Type
Enterprise server image for this PR: