[Fix] UI - Hide Default Team Settings From Proxy Admin Viewers #16900
+316
−43
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.
[Fix] UI - Hide Default Team Settings From Proxy Admin Viewers
Relevant issues
This PR hides the Default Team Settings Tab from Proxy Admin Viewers. Judging from the design, this tab was never intended to be seen by Proxy Admin Viewers as /get/default_team_settings will throw an error if you are not a proxy admin
This was causing some confusion among users because an error Notification stating "Failed to fetch team settings" will show up. The root cause is inside Default Team Settings, there is network call to /get/default_team_settings, only for it to return with an error, triggering
litellm/ui/litellm-dashboard/src/components/TeamSSOSettings.tsx
Line 51 in 7225fc0
Pre-Submission checklist
Please complete all items before asking a LiteLLM maintainer to review your PR
tests/litellm/directory, Adding at least 1 test is a hard requirement - see detailsmake test-unitType
🐛 Bug Fix
✅ Test
Changes