chore: remove frontend scim checks for user management#5575
Conversation
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
Greptile SummaryThis PR removes frontend-only SCIM guards that were previously blocking user deletion, deactivation, and activation at the organization level when Key changes:
No new logic is introduced, no imports are left dangling, and Confidence Score: 5/5
Last reviewed commit: 97caf90 |
Additional Comments (1)
This PR removes SCIM checks from the individual "Remove User" and "Activate/Deactivate User" buttons in // OrgMembersSection.tsx — still present after this PR
isDisabled={!isAllowed || currentOrg?.scimEnabled}This creates a confusing inconsistency: admins can remove individual members via the per-row dropdown even when SCIM is enabled, but the bulk-delete button remains blocked. If the intent is to lift SCIM restrictions on user management entirely, the bulk-delete guard in |
|
@greptile re-review this PR |
Context
This PR removes the unnecessary SCIM checks during user deletion at the organization level
Screenshots
Steps to verify the change
Type
Checklist
type(scope): short description(scope is optional, e.g.,fix: prevent crash on syncorfix(api): handle null response).