Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Easier user onboarding via SSO #8187

Merged
merged 6 commits into from
Feb 3, 2025
Merged

Conversation

krrishdholakia
Copy link
Contributor

@krrishdholakia krrishdholakia commented Feb 1, 2025

Title

  • Allows finding users by their email, and attaching the sso user id to the user if found
  • Makes separate team page
  • enables proxy / team admin to delete members
  • separates user id from user email
  • adds auto-suggest when adding user id / user email on team member add
Screen.Recording.2025-02-01.at.8.04.36.PM.mov

Relevant issues

Reduces scope for bugs caused by previous flow of updating the user id to the sso user id

Updated flow - now checks for user email match -> stores sso_user_id in the new sso_user_id column for that user, so we can refer back to this mapping.

Closes #6190

Type

🐛 Bug Fix
🧹 Refactoring

Changes

fix(ui_sso.py): use common get_user_object logic across jwt + ui sso auth

[REQUIRED] Testing - Attach a screenshot of any new tests passing locally

If UI changes, send a screenshot/GIF of working UI fixes

…o auth

Allows finding users by their email, and attaching the sso user id to the user if found
Copy link

vercel bot commented Feb 1, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
litellm ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 3, 2025 6:53am

* build(teams.tsx): refactor teams page to make it easier to add members to a team

make a row in table clickable -> allows user to add users to team they intended

* build(teams.tsx): make it clear user should click on team id to view team details

simplifies team management by putting team details on separate page

* build(team_info.tsx): separately show user id and user email

make it easy for user to understand the information they're seeing

* build(team_info.tsx): add back in 'add member' button

* build(team_info.tsx): working team member update on team_info.tsx

* build(team_info.tsx): enable team member delete on ui

allow user to delete accidental adds

* build(internal_user_endpoints.py): expose new endpoint for ui to allow filtering on user table

allows proxy admin to quickly find user they're looking for

* feat(team_endpoints.py): expose new team filter endpoint for ui

allows proxy admin to easily find team they're looking for

* feat(user_search_modal.tsx): allow admin to filter on users when adding new user to teams
@krrishdholakia krrishdholakia merged commit 6834c5e into main Feb 3, 2025
26 of 31 checks passed
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.

[Feature]: Canonicalize the Member Name in the Team Members display
1 participant